Task #143 - pinentry-1.3.0-2.fc41/scan-results.err
back to task #143download
Error: GCC_ANALYZER_WARNING (CWE-401): pinentry-1.3.0/gnome3/pinentry-gnome3.c: scope_hint: In function ‘_propagate_g_error_to_pinentry’ pinentry-1.3.0/gnome3/pinentry-gnome3.c:111:7: warning[-Wanalyzer-malloc-leak]: leak of ‘*pe.specific_err_info’ # 109| * no need to check for failure. */ # 110| pe->specific_err_info = strdup (t); # 111|-> g_free (t); # 112| } # 113| else Error: GCC_ANALYZER_WARNING (CWE-401): pinentry-1.3.0/pinentry/pinentry-curses.c: scope_hint: In function ‘pinentry_utf8_to_local’ pinentry-1.3.0/pinentry/pinentry-curses.c:220:6: warning[-Wanalyzer-malloc-leak]: leak of ‘iconv_open(target_encoding, "UTF-8")’ pinentry-1.3.0/pinentry/pinentry-curses.c:607:3: note: in expansion of macro ‘COPY_OUT’ pinentry-1.3.0/pinentry/pinentry-curses.c:607:3: note: in expansion of macro ‘COPY_OUT’ pinentry-1.3.0/pinentry/pinentry-curses.c:607:3: note: in expansion of macro ‘COPY_OUT’ # 218| # 219| cd = iconv_open (target_encoding, "UTF-8"); # 220|-> if (cd == (iconv_t) -1) # 221| { # 222| fprintf (stderr, "%s: can't convert from UTF-8 to %s: %s\n", Error: CLANG_WARNING: pinentry-1.3.0/pinentry/pinentry-curses.c:522:7: warning[deadcode.DeadStores]: Value stored to 'i' is never read # 520| p++; # 521| # 522|-> i = 0; # 523| error_y++; # 524| } Error: CLANG_WARNING: pinentry-1.3.0/pinentry/pinentry-curses.c:1775:12: warning[deadcode.DeadStores]: Although the value stored to 'tim' is used in the enclosing expression, the value is never actually read from 'tim' # 1773| # 1774| /* Make sure that we actually update the mtime. */ # 1775|-> while ( (tim = time (NULL)) == st.st_mtime ) # 1776| sleep (1); # 1777| Error: CPPCHECK_WARNING (CWE-476): pinentry-1.3.0/pinentry/pinentry-emacs.c:186: error[nullPointer]: Null pointer dereference: (struct sockaddr_un*)0 # 184| # 185| if (connect (emacs_socket, (struct sockaddr *) &unaddr, # 186|-> SUN_LEN (&unaddr)) < 0) # 187| { # 188| perror ("connect"); Error: CLANG_WARNING: pinentry-1.3.0/pinentry/pinentry-emacs.c:590:12: warning[deadcode.DeadStores]: Although the value stored to 'tim' is used in the enclosing expression, the value is never actually read from 'tim' # 588| # 589| /* Make sure that we actually update the mtime. */ # 590|-> while ( (tim = time (NULL)) == st.st_mtime ) # 591| sleep (1); # 592| Error: CLANG_WARNING: pinentry-1.3.0/pinentry/pinentry.c:987:13: warning[deadcode.DeadStores]: Value stored to 'tmpstr' is never read # 985| tmpstr = parse_color (tmpstr, &pinentry.color_ok, # 986| &pinentry.color_ok_bright); # 987|-> tmpstr = parse_color (tmpstr, &pinentry.color_qualitybar, # 988| &pinentry.color_qualitybar_bright); # 989| } Error: GCC_ANALYZER_WARNING (CWE-465): pinentry-1.3.0/pinentry/pinentry.c: scope_hint: In function ‘option_handler’ pinentry-1.3.0/pinentry/pinentry.c:1119:18: warning[-Wanalyzer-deref-before-check]: check of ‘endp’ for NULL after already dereferencing it # 1117| pinentry.owner_uid = (int)along; # 1118| } # 1119|-> if (endp) # 1120| { # 1121| while (*endp == ' ') Error: GCC_ANALYZER_WARNING (CWE-476): pinentry-1.3.0/pinentry/pinentry.c:1127:28: warning[-Wanalyzer-possible-null-dereference]: dereference of possibly-NULL ‘endp’ # 1125| pinentry.owner_host = strdup (endp); # 1126| for (endp=pinentry.owner_host; # 1127|-> *endp && *endp != ' '; endp++) # 1128| ; # 1129| *endp = 0; Error: CLANG_WARNING: pinentry-1.3.0/pinentry/pinentry.c:1987:12: warning[deadcode.DeadStores]: Although the value stored to 'j' is used in the enclosing expression, the value is never actually read from 'j' # 1985| gpg_error_t rc; # 1986| # 1987|-> for (i = j = 0; table[i].name; i++) # 1988| { # 1989| rc = assuan_register_command (ctx, table[i].name, table[i].handler, NULL); Error: GCC_ANALYZER_WARNING (CWE-688): pinentry-1.3.0/secmem/secmem.c: scope_hint: In function ‘secmem_realloc’ pinentry-1.3.0/secmem/secmem.c:365:5: warning[-Wanalyzer-null-argument]: use of NULL where non-null expected <built-in>: note: argument 1 of ‘__builtin_memcpy’ must be non-null # 363| return p; /* it is easier not to shrink the memory */ # 364| a = secmem_malloc( newsize ); # 365|-> memcpy(a, p, size); # 366| memset((char*)a+size, 0, newsize-size); # 367| secmem_free(p); Error: CLANG_WARNING: pinentry-1.3.0/tty/pinentry-tty.c:503:11: warning[deadcode.DeadStores]: Although the value stored to 'tim' is used in the enclosing expression, the value is never actually read from 'tim' # 501| # 502| /* Make sure that we actually update the mtime. */ # 503|-> while ((tim = time(NULL)) == st.st_mtime) # 504| sleep(1); # 505|