psmisc-23.7-1.fc41

List of Defects

Error: GCC_ANALYZER_WARNING (CWE-401): [#def1]
psmisc-23.7/src/fuser.c: scope_hint: In function ‘add_matched_proc’
psmisc-23.7/src/fuser.c:442:8: warning[-Wanalyzer-malloc-leak]: leak of ‘pptr’
#  440|       if (pathname)
#  441|           free(pathname);
#  442|->     if (fp)
#  443|           fclose(fp);
#  444|   }

Error: CLANG_WARNING: [#def2]
psmisc-23.7/src/fuser.c:2165:18: warning[deadcode.DeadStores]: Although the value stored to 'end' is used in the enclosing expression, the value is never actually read from 'end'
# 2163|       }
# 2164|   
# 2165|->     for (start = end = path; *start; start = end)
# 2166|       {
# 2167|           while (*start == '/')

Error: CLANG_WARNING: [#def3]
psmisc-23.7/src/fuser.c:2222:13: warning[deadcode.DeadStores]: Value stored to 'path' is never read
# 2220|   
# 2221|               memmove(&tmpbuf[n], end, len + 1);
# 2222|->             path = end = memcpy(tmpbuf, lnkbuf, n);
# 2223|   
# 2224|               if (lnkbuf[0] == '/')

Error: CLANG_WARNING: [#def4]
psmisc-23.7/src/killall.c:305:2: warning[core.CallAndMessage]: Called function pointer is null (null dereference)
#  303|       if (my_getpidcon && selinux_enabled && !my_getpidcon(pid, &lcontext)) {
#  304|           retval = (regexec(scontext, lcontext, 0, NULL, 0) ==0);
#  305|-> 	my_freecon(lcontext);
#  306|       } else {
#  307|           FILE *file;

Error: GCC_ANALYZER_WARNING (CWE-401): [#def5]
psmisc-23.7/src/killall.c: scope_hint: In function ‘load_process_name_and_age’
psmisc-23.7/src/killall.c:414:16: warning[-Wanalyzer-malloc-leak]: leak of ‘file’
#  412|   
#  413|       if ( (fd = openat(pidfd, "stat", O_RDONLY, 0)) < 0)
#  414|->         return -1;
#  415|       if (!(file = fdopen (fd, "r")))
#  416|       {

Error: GCC_ANALYZER_WARNING (CWE-457): [#def6]
psmisc-23.7/src/killall.c: scope_hint: In function ‘load_proc_cmdline’
psmisc-23.7/src/killall.c:503:33: warning[-Wanalyzer-use-of-uninitialized-value]: use of uninitialized value ‘*command_buf’
#  501|               }
#  502|           }
#  503|->         if (strlen(command_buf) == 0) {
#  504|               okay = 0;
#  505|               break;

Error: GCC_ANALYZER_WARNING (CWE-401): [#def7]
psmisc-23.7/src/killall.c:530:15: warning[-Wanalyzer-malloc-leak]: leak of ‘file’
#  528|           return -1;
#  529|       }
#  530|->     *got_long = okay;
#  531|       return 0;
#  532|   }

Error: GCC_ANALYZER_WARNING (CWE-401): [#def8]
psmisc-23.7/src/killall.c: scope_hint: In function ‘create_pid_table’
psmisc-23.7/src/killall.c:550:8: warning[-Wanalyzer-malloc-leak]: leak of ‘opendir("/proc")’
#  548|       *max_pids = 256;
#  549|       pid_table = malloc (*max_pids * sizeof (pid_t));
#  550|->     if (!pid_table)
#  551|       {
#  552|           perror ("malloc");

Error: GCC_ANALYZER_WARNING (CWE-401): [#def9]
psmisc-23.7/src/killall.c: scope_hint: In function ‘kill_all’
psmisc-23.7/src/killall.c:558:37: warning[-Wanalyzer-malloc-leak]: leak of ‘create_pid_table(&max_pids, & pids)’
#  556|       while ( (de = readdir (dir)) != NULL)
#  557|       {
#  558|->         if (!(pid = (pid_t) atoi (de->d_name)) || pid == self)
#  559|               continue;
#  560|           if (*pids == *max_pids)

Error: CLANG_WARNING: [#def10]
psmisc-23.7/src/killall.c:758:22: warning[core.NullDereference]: Array access (from variable 'pgids') results in a null pointer dereference
#  756|   
#  757|               id = getpgid (pid_table[i]);
#  758|->             pgids[i] = id;
#  759|               if (id < 0)
#  760|               {

Error: CLANG_WARNING: [#def11]
psmisc-23.7/src/killall.c:800:32: warning[core.BitwiseShift]: Right operand is negative in left shift
#  798|           /* killall returns a zero return code if at least one process has 
#  799|            * been killed for each listed command. */
#  800|->         error = found == ((1UL << (name_count - 1)) | ((1UL << (name_count - 1)) - 1)) ? 0 : 1;
#  801|       else
#  802|           /* in nameless mode killall returns a zero return code if at least 

Error: CLANG_WARNING: [#def12]
psmisc-23.7/src/killall.c:938:9: warning[deadcode.DeadStores]: Value stored to 'name' is never read
#  936|           name++;
#  937|       else
#  938|->         name = *argv;
#  939|       sig_num = SIGTERM;
#  940|   

Error: GCC_ANALYZER_WARNING (CWE-479): [#def13]
psmisc-23.7/src/peekfd.c: scope_hint: In function ‘detach’
psmisc-23.7/src/peekfd.c:173:17: warning[-Wanalyzer-unsafe-call-within-signal-handler]: call to ‘free’ from within signal handler
#  171|   		ptrace(PTRACE_DETACH, attached_pids[i], 0, 0);
#  172|   	if (fds)
#  173|-> 		free(fds);
#  174|   	signal(SIGINT, SIG_DFL);
#  175|   	raise(SIGINT);

Error: CPPCHECK_WARNING (CWE-401): [#def14]
psmisc-23.7/src/pstree.c:374: error[memleakOnRealloc]: Common realloc mistake: 'width' nulled but not freed upon failure
#  372|           else
#  373|               capacity *= 2;
#  374|->         if (!(width = realloc(width, capacity * sizeof(int)))) {
#  375|               perror("realloc");
#  376|               exit(1);

Error: CPPCHECK_WARNING (CWE-401): [#def15]
psmisc-23.7/src/pstree.c:378: error[memleakOnRealloc]: Common realloc mistake: 'more' nulled but not freed upon failure
#  376|               exit(1);
#  377|           }
#  378|->         if (!(more = realloc(more, capacity * sizeof(int)))) {
#  379|               perror("realloc");
#  380|               exit(1);

Error: CLANG_WARNING: [#def16]
psmisc-23.7/src/pstree.c:521:9: warning[core.CallAndMessage]: Called function pointer is null (null dereference)
#  519|       if (my_getpidcon && selinux_enabled && !my_getpidcon(current->pid, &context)) {
#  520|           out_string(context);
#  521|->         my_freecon(context);
#  522|           ret = true;
#  523|       }

Error: CLANG_WARNING: [#def17]
psmisc-23.7/src/pstree.c:918:9: warning[deadcode.DeadStores]: Value stored to 'count' is never read
#  916|       {
#  917|           width[level] = swapped + (comm_len > 1 ? 0 : -1);
#  918|->         count=0;
#  919|           first=1;
#  920|           for (walk = current->children; walk; walk = next) {

Error: CLANG_WARNING: [#def18]
psmisc-23.7/src/pstree.c:919:9: warning[deadcode.DeadStores]: Value stored to 'first' is never read
#  917|           width[level] = swapped + (comm_len > 1 ? 0 : -1);
#  918|           count=0;
#  919|->         first=1;
#  920|           for (walk = current->children; walk; walk = next) {
#  921|             next = walk->next;

Error: GCC_ANALYZER_WARNING (CWE-775): [#def19]
psmisc-23.7/src/pstree.c: scope_hint: In function ‘read_proc’
psmisc-23.7/src/pstree.c:1176:33: warning[-Wanalyzer-file-leak]: leak of FILE ‘fopen(path, "r")’
psmisc-23.7/src/pstree.c:41: included_from: Included from here.
# 1174|                 int thread;
# 1175|   
# 1176|-> 	      process_age_sec = process_age(proc_stt_jf);
# 1177|                 /* handle process threads */
# 1178|                 if (! hide_threads) {

Error: GCC_ANALYZER_WARNING (CWE-401): [#def20]
psmisc-23.7/src/pstree.c:1176:33: warning[-Wanalyzer-malloc-leak]: leak of ‘fopen(path, "r")’
# 1174|                 int thread;
# 1175|   
# 1176|-> 	      process_age_sec = process_age(proc_stt_jf);
# 1177|                 /* handle process threads */
# 1178|                 if (! hide_threads) {

Error: CLANG_WARNING: [#def21]
psmisc-23.7/src/pstree.c:1237:34: warning[core.NullDereference]: Array access (from variable 'buffer') results in a null pointer dereference
# 1235|                     size--;
# 1236|                   if (size)
# 1237|->                   buffer[size++] = 0;
# 1238|                   add_proc(comm, pid, ppid, pgid, st.st_uid,
# 1239|                        buffer, size, 0, process_age_sec);

Error: CLANG_WARNING: [#def22]
psmisc-23.7/src/statx.c:63:11: warning[core.NonNullParamChecker]: Null pointer passed to 2nd parameter expecting 'nonnull'
#   61|       ret = syscall(SYS_statx, dirfd, pathname, flags, mask, &stx);
#   62|   #else
#   63|->     ret = statx(dirfd, pathname, flags, mask, &stx);
#   64|   #endif
#   65|       if (ret >= 0) {

Error: CLANG_WARNING: [#def23]
psmisc-23.7/src/statx.c:134:11: warning[core.NonNullParamChecker]: Null pointer passed to 2nd parameter expecting 'nonnull'
#  132|       ret = syscall(SYS_statx, dirfd, pathname, flags, mask, &stx);
#  133|   #else
#  134|->     ret = statx(dirfd, pathname, flags, mask, &stx);
#  135|   #endif
#  136|       if (ret >= 0) {

Scan Properties