]> granicus.if.org Git - procps-ng/commit
Fix readlink's do-while-loop
authorTobias Stoeckmann <tobias@stoeckmann.org>
Sat, 11 Jul 2015 19:30:31 +0000 (21:30 +0200)
committerTobias Stoeckmann <tobias@stoeckmann.org>
Sat, 11 Jul 2015 19:30:31 +0000 (21:30 +0200)
commite8430bad0057c8880f2f453ea90b8414ed30d0d5
tree93ef7031b4a0de342e38920d18008c5ca8d12a07
parent30986cb22e2f26d83cbb8516bf25d3fa5c9a577d
Fix readlink's do-while-loop

The function pid_link tries to handle programs which contain very
long paths to their executables. If 1024 bytes are not enough to
contain the path, the loop wants to get more and more space until
the path can fit.

The loop's condition does not fit though.

readlink will never return a value higher than its supplied size
limit, which is "path_alloc_size - 1", therefore the loop-check of
"len == path_alloc_size" will always be false: the loop will never
be repeated.

While at it, the if-condition inside the loop's body can be omitted,
because it is always true.
pidof.c