feedc0de/wolfssl
1
0
Fork 0
forked from wolfSSL/wolfssl
Code Pull Requests Activity

Merge pull request #8031 from douzzer/20240930-clang-tidy

Browse Source 
20240930-clang-tidy
This commit is contained in:
Sean Parkinson
2024-10-01 22:38:18 +10:00
committed by GitHub
parent 984dd9146f 65853a41b9
commit f15700f1f6
4 changed files with 21 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
examples/pem/pem.c
View File

@ -1024,6 +1024,13 @@ int main(int argc, char* argv[])
if (ret < 0) { if (ret < 0) {
fprintf(stderr, "%s\n", wc_GetErrorString(ret)); fprintf(stderr, "%s\n", wc_GetErrorString(ret));
} }
if (in_file != stdin)
(void)fclose(in_file);
if (out_file != stdout)
(void)fclose(out_file);
return (ret == 0) ? 0 : 1; return (ret == 0) ? 0 : 1;
} }

6
src/ssl.c
View File

@ -23999,7 +23999,7 @@ int wolfSSL_RAND_seed(const void* seed, int len)
*/ */
const char* wolfSSL_RAND_file_name(char* fname, unsigned long len) const char* wolfSSL_RAND_file_name(char* fname, unsigned long len)
{ {
#if !defined(NO_FILESYSTEM) && defined(XGETENV) #if !defined(NO_FILESYSTEM) && defined(XGETENV) && !defined(NO_GETENV)
char* rt; char* rt;
WOLFSSL_ENTER("wolfSSL_RAND_file_name"); WOLFSSL_ENTER("wolfSSL_RAND_file_name");
@ -24010,6 +24010,7 @@ const char* wolfSSL_RAND_file_name(char* fname, unsigned long len)
XMEMSET(fname, 0, len); XMEMSET(fname, 0, len);
/* // NOLINTBEGIN(concurrency-mt-unsafe) */
if ((rt = XGETENV("RANDFILE")) != NULL) { if ((rt = XGETENV("RANDFILE")) != NULL) {
if (len > XSTRLEN(rt)) { if (len > XSTRLEN(rt)) {
XMEMCPY(fname, rt, XSTRLEN(rt)); XMEMCPY(fname, rt, XSTRLEN(rt));
@ -24019,6 +24020,7 @@ const char* wolfSSL_RAND_file_name(char* fname, unsigned long len)
rt = NULL; rt = NULL;
} }
} }
/* // NOLINTEND(concurrency-mt-unsafe) */
/* $RANDFILE was not set or is too large, check $HOME */ /* $RANDFILE was not set or is too large, check $HOME */
if (rt == NULL) { if (rt == NULL) {
@ -24026,6 +24028,7 @@ const char* wolfSSL_RAND_file_name(char* fname, unsigned long len)
WOLFSSL_MSG("Environment variable RANDFILE not set"); WOLFSSL_MSG("Environment variable RANDFILE not set");
/* // NOLINTBEGIN(concurrency-mt-unsafe) */
if ((rt = XGETENV("HOME")) == NULL) { if ((rt = XGETENV("HOME")) == NULL) {
#ifdef XALTHOMEVARNAME #ifdef XALTHOMEVARNAME
if ((rt = XGETENV(XALTHOMEVARNAME)) == NULL) { if ((rt = XGETENV(XALTHOMEVARNAME)) == NULL) {
@ -24038,6 +24041,7 @@ const char* wolfSSL_RAND_file_name(char* fname, unsigned long len)
return NULL; return NULL;
#endif #endif
} }
/* // NOLINTEND(concurrency-mt-unsafe) */
if (len > XSTRLEN(rt) + XSTRLEN(ap)) { if (len > XSTRLEN(rt) + XSTRLEN(ap)) {
fname[0] = '\0'; fname[0] = '\0';

8
src/ssl_load.c
View File

@ -5099,7 +5099,7 @@ int wolfSSL_CTX_use_RSAPrivateKey(WOLFSSL_CTX* ctx, WOLFSSL_RSA* rsa)
int wolfSSL_CTX_set_default_verify_paths(WOLFSSL_CTX* ctx) int wolfSSL_CTX_set_default_verify_paths(WOLFSSL_CTX* ctx)
{ {
int ret; int ret;
#ifdef XGETENV #if defined(XGETENV) && !defined(NO_GETENV)
char* certDir = NULL; char* certDir = NULL;
char* certFile = NULL; char* certFile = NULL;
word32 flags = 0; word32 flags = 0;
@ -5109,7 +5109,8 @@ int wolfSSL_CTX_set_default_verify_paths(WOLFSSL_CTX* ctx)
WOLFSSL_ENTER("wolfSSL_CTX_set_default_verify_paths"); WOLFSSL_ENTER("wolfSSL_CTX_set_default_verify_paths");
#ifdef XGETENV #if defined(XGETENV) && !defined(NO_GETENV)
/* // NOLINTBEGIN(concurrency-mt-unsafe) */
certDir = wc_strdup_ex(XGETENV("SSL_CERT_DIR"), DYNAMIC_TYPE_TMP_BUFFER); certDir = wc_strdup_ex(XGETENV("SSL_CERT_DIR"), DYNAMIC_TYPE_TMP_BUFFER);
certFile = wc_strdup_ex(XGETENV("SSL_CERT_FILE"), DYNAMIC_TYPE_TMP_BUFFER); certFile = wc_strdup_ex(XGETENV("SSL_CERT_FILE"), DYNAMIC_TYPE_TMP_BUFFER);
flags = WOLFSSL_LOAD_FLAG_PEM_CA_ONLY; flags = WOLFSSL_LOAD_FLAG_PEM_CA_ONLY;
@ -5133,6 +5134,7 @@ int wolfSSL_CTX_set_default_verify_paths(WOLFSSL_CTX* ctx)
ret = 0; ret = 0;
} }
} }
/* // NOLINTEND(concurrency-mt-unsafe) */
else else
#endif #endif
@ -5157,7 +5159,7 @@ int wolfSSL_CTX_set_default_verify_paths(WOLFSSL_CTX* ctx)
#endif #endif
} }
#ifdef XGETENV #if defined(XGETENV) && !defined(NO_GETENV)
XFREE(certFile, NULL, DYNAMIC_TYPE_TMP_BUFFER); XFREE(certFile, NULL, DYNAMIC_TYPE_TMP_BUFFER);
XFREE(certDir, NULL, DYNAMIC_TYPE_TMP_BUFFER); XFREE(certDir, NULL, DYNAMIC_TYPE_TMP_BUFFER);
#endif #endif

8
wolfssl/openssl/asn1.h
View File

@ -270,8 +270,8 @@ typedef struct WOLFSSL_ASN1_ITEM WOLFSSL_ASN1_ITEM;
(WolfsslAsn1FreeCb)member_type##_free, \ (WolfsslAsn1FreeCb)member_type##_free, \
(WolfsslAsn1i2dCb)i2d_##member_type, \ (WolfsslAsn1i2dCb)i2d_##member_type, \
(WolfsslAsn1d2iCb)d2i_##member_type, \ (WolfsslAsn1d2iCb)d2i_##member_type, \
0, flags & ASN1_TFLG_TAG_MASK ? tag : -1, 0, \ 0, (flags) & ASN1_TFLG_TAG_MASK ? (tag) : -1, 0, \
!!(flags & ASN1_TFLG_EXPLICIT), TRUE } !!((flags) & ASN1_TFLG_EXPLICIT), TRUE }
WOLFSSL_API void *wolfSSL_ASN1_item_new(const WOLFSSL_ASN1_ITEM *tpl); WOLFSSL_API void *wolfSSL_ASN1_item_new(const WOLFSSL_ASN1_ITEM *tpl);
WOLFSSL_API void wolfSSL_ASN1_item_free(void *obj, WOLFSSL_API void wolfSSL_ASN1_item_free(void *obj,
@ -282,7 +282,7 @@ WOLFSSL_API void* wolfSSL_ASN1_item_d2i(void** dst, const byte **src, long len,
const WOLFSSL_ASN1_ITEM* item); const WOLFSSL_ASN1_ITEM* item);
/* Need function declaration otherwise compiler complains */ /* Need function declaration otherwise compiler complains */
/* // NOLINTBEGIN(readability-named-parameter) */ /* // NOLINTBEGIN(readability-named-parameter,bugprone-macro-parentheses) */
#define IMPLEMENT_ASN1_FUNCTIONS(type) \ #define IMPLEMENT_ASN1_FUNCTIONS(type) \
type *type##_new(void); \ type *type##_new(void); \
type *type##_new(void){ \ type *type##_new(void){ \
@ -303,7 +303,7 @@ WOLFSSL_API void* wolfSSL_ASN1_item_d2i(void** dst, const byte **src, long len,
return (type*)wolfSSL_ASN1_item_d2i((void**)dst, src, len, \ return (type*)wolfSSL_ASN1_item_d2i((void**)dst, src, len, \
&type##_template_data); \ &type##_template_data); \
} }
/* // NOLINTEND(readability-named-parameter) */ /* // NOLINTEND(readability-named-parameter,bugprone-macro-parentheses) */
#endif /* OPENSSL_ALL */ #endif /* OPENSSL_ALL */