Skip to content

fix uid/gid > 2^31

Todd Lewis requested to merge utoddl/procps:or-large-uid into master

This MR revisits a partial fix from 2018. The problem stems from incorrect handling of unsigned 32-bit uid_ts and gid_ts as signed when values are large - i.e. when the high bit is set. In that case, pgrep and pkill fail to identify processes by uid. (They succeed when finding the same processes by username.) The primary fix for this is to impliment the "FIXME" comment in proc/readproc.h, the implementation of which allows the removal of the (int) casts from the partial fix from 2018.

The other fixed code in this MR consists of tests in strict_atol() that detects and errors out on overflows.

Merge request reports