doc: fix typo in kill1ll.1

Lev reported that we were saying in the manpage that killall uses
expended regular expression and not exTended regular expression.

fuser: -k return status based on any found

In [bugs:#66] the ever-helpful "Anonymous" reported that
fuser with -k would only return correctly if the process
was found last and not if it was found anywhere.

While I have solved this problem a different way, the report
was enough to narrow down what was going wrong. Now fuser -k
returns 1 if it finds any process to kill, not just if the last
one was; just like fuser with no -k works.

Signed-off-by: Craig Small <csmall@enc.com.au>

Avoid possible crash and memory leak in killall

By reading the current HEAD I stumbled on a memory leak in killall.c
as well as a not assigned pointer got_long in load_proc_cmdline()
which may lead to SIGSEGV.

References: [patches:#67] https://sourceforge.net/p/psmisc/bugs/67
Signed-off-by: Werner Fink <werner@suse.de>

Fixed typo in fuser.1 Debian #763374

Updated translations

Imported the latest po files from the translation
project.

killall younger and older flags

The -y and -o flags were not being used if you used the
regex (-r) flag.  I initially moved those flags to the
right part of the loop.

However looking at that giant loop of code it was very difficult
to debug so I have also taken the opportunity to re-work it
with some functions, so its clearer what is going on.

Removed early free on name_len

name_len was being freed too early and in a loop so
sometimes you would hit a double-free problem
The locale changes were to keep efence happy though
i suspect something strange was going on with locales there
too.

Include <limits.h> for PATH_MAX

When building against musl libc, PATH_MAX is defined in <limits.h>.

Signed-off-by: Paul Barker <paul@paulbarker.me.uk>
Signed-off-by: Craig Small <csmall@enc.com.au>

Note S patch #34 for pstree

pstree: Ignore processes that disappear before reading cmdline

The worst case here is that the user has specific a PID on the
command-line, read_proc() fails to read cmdline for an irrelevant,
transient process and fails.

It is better to simply ignore processes that have disappeared.  That
way pstree can provide useful output in more cases.

An alternative is to read cmdline only for processes that the user is
interested in.  However, this probably increases the chances of an
interesting process exiting before the cmdline is read.

Signed-off-by: Martin Schwenke <martin@meltin.net>

define PATH_MAX for freebsd

Fix some segfaults due to freeing too early

Fixed v4 versus v6 problem

fuser -n tcp 80 always worked because the v4 or v6 problem didnt hit
single ports, but any ipv4 or ipv6 remote address wouldn't work.
Caught by Coverity CID46269

Update translations

Minor fixes due to Coverity reporting

Merge branch 'master' of ssh://git.code.sf.net/p/psmisc/code

Full thread display for pstree an option

Commit [9de89a] made pstree show the correct thread names. This had
the unintended consequence of making a lot of thread views
uncompact, as the names would be different.

Allan suggested that if you want to see the real thread names, then
explictly ask for them. I agree and pstree now has a new option -t
for showing thread names.

$ pstree 22603
udisksd───4*[{udisksd}]
$ pstree -t 22603
udisksd─┬─{cleanup}
        ├─{gdbus}
        ├─{gmain}
        └─{probing-thread}

References: [patches:#33] https://sourceforge.net/p/psmisc/patches/33/
Signed-off-by: Craig Small <csmall@enc.com.au>

Full thread display for pstree an option

Commit [9de89a] made pstree show the correct thread names. This had
the unintended consequence of making a lot of thread views
uncompact, as the names would be different.

Allan suggested that if you want to see the real thread names, then
explictly ask for them. I agree and pstree now has a new option -t
for showing thread names.

$ pstree 22603
udisksd───4*[{udisksd}]
$ pstree -t 22603
udisksd─┬─{cleanup}
        ├─{gdbus}
        ├─{gmain}
        └─{probing-thread}

References: [#33] https://sourceforge.net/p/psmisc/patches/33/
Signed-off-by: Craig Small <csmall@enc.com.au>

Typo in fuser makes -M on all the time

Brad found that fuser had the -M option on all the time.
A simple but significant typo caused this, thanks the the patch.

Bug-Debian: http://bugs.debian.org/740275

Signed-off-by: Craig Small <csmall@enc.com.au>

Updated po files

Signed-off-by: Craig Small <csmall@enc.com.au>

Last translation update for this version

Updated the ones that came in later.

Signed-off-by: Craig Small <csmall@enc.com.au>

Update the translations

Signed-off-by: Craig Small <csmall@enc.com.au>

Another PO file update

Signed-off-by: Craig Small <csmall@enc.com.au>

Update po files from TP site

rsyned the latest po files from TP site, ready for the last update.

Signed-off-by: Craig Small <csmall@enc.com.au>

pstree: fclose fopened files

Commit [9de89a] uses fopen to find the tasks, but doesn't fclose
them afterward. Thanks for the anonymous and somewhat obscure
bug commenter for pointing it out.

Signed-off-by: Craig Small <csmall@enc.com.au>

fuser -M and -m flags work either way

Previously fuser -M -m /dir worked while fuser -m /dir -M would
not limit the search. This change means either way will work.

References: http://bugs.debian.org/606178

Signed-off-by: Craig Small <csmall@enc.com.au>

Port peekfd to m68k

Fixing null pointer dereference

Introduced with the namespaces support

References:
  https://sourceforge.net/p/psmisc/patches/32/
  https://sourceforge.net/p/psmisc/code/ci/e64b282643afecb7060c321f8401872feaf07dbb/log/?path=

Signed-off-by: Craig Small <csmall@enc.com.au>

Fixed -p -g typo in psmisc.1

The explanation for -g had an errant -p in the description.

References: https://sourceforge.net/p/psmisc/bugs/61/

Update for 22.21

Updated the translation files
Minor fix in killall man page

pstree shows threads correct names.

pstree would only show the process name, not the thread name. This
change now makes pstree read /proc/<PID>/task/<TID>/stat to find the
threads' real names.
$ pstree -p 1187
udisksd(1187)─┬─{udisksd}(1189)
              ├─{udisksd}(1191)
              ├─{udisksd}(1192)
              ├─{udisksd}(1195)
              └─{udisksd}(27395)
./pstree -p 1187
udisksd(1187)─┬─{cleanup}(1195)
              ├─{gdbus}(1191)
              ├─{gmain}(1189)
              ├─{pool}(27395)
              └─{probing-thread}(1192)

References:
  https://sourceforge.net/p/psmisc/bugs/57/

This fix was made a lot easier due to Mapk providing a small test script
showing the broken behaviour, thanks!

Update of the po/pot files

pstree: introduce namespace transition information

This patch adds a new option (-S, --ns-change) that will show
when a namespace was changed compared to parent's.

Signed-off-by: Aristeu Rozanski <arozansk@redhat.com>
Signed-off-by: Craig Small <csmall@enc.com.au>

pstree: introduce support for namespaces

Options -N and --ns-sort were added which require one of the namespaces:
ipc, mnt, net, pid, user, uts
and will show separated trees per namespace

Signed-off-by: Aristeu Rozanski <arozansk@redhat.com>
Signed-off-by: Craig Small <csmall@enc.com.au>

pstree uses COLUMNS environement variable

pstree previously only used the window size for determining number of
columns, then a default of 132. With this change, pstree now checks
the COLUMNS environment variable first and uses that if valid.

env checking code nicked from top.c, by Jim Warner.

Bug-Debian: http://bugs.debian.org/717017

Added missing comma from fuser(1)

Actuallly it was a missing R because \f, shows nothing but \fR, shows a
comma.
Bug-Debian: http://bugs.debian.org/702391

ignore tar.gz and config directory

doesn't complain about "not a directory"

If /proc/#/fd points to a path which has component removed, fuser
will complain like "Cannot stat file /proc/29322/fd/16: Not a
directory". Looks like it can be disabled just like commit 898dc70a1f
did.

Signed-off-by: Craig Small <csmall@enc.com.au>

Update po files

use AC_CANONICAL_HOST

--host is used for the system where the compiled code will run.
--target is not the correct system to look that up.
http://www.gnu.org/software/autoconf/manual/html_node/Manual-Configuration.html#Manual-Configuration

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Craig Small <csmall@enc.com.au>

pstree compiles with SE Linux

pstree failed to compile with SE Linux enabled because one of the
scontext was missed and without it enabled the bug doesn't appear.

pstree is now re-worked so scontext is defined as a dummy meaning
most of the code except the reall SE Linux specific stuff is compilied
meaning this sort of thing shouldn't happen too much again.

Bug-Gentoo: https://bugs.gentoo.org/show_bug.cgi?id=437332
Bug-Sourceforge: https://sourceforge.net/p/psmisc/bugs/54/

updated with distcheck

update translations

freebsd pid 0 and no loops

pstree looped forever and never showed anything below pid 1
this change fixes the head of the tree to make sure it has a null
pointer for a parent

Signed-off-by: Craig Small <csmall@enc.com.au>

Updated po files

force process age >= 0

sometimes some small numbers would hit an assert around process age,
this fix just makes really young processes have an age of 0

Bug-Debian: http://bugs.debian.org/628617

fuser finds unix sockets

in check_dir() stat was replaced with device() which found the right
device but zeroed out the inode, making that check always fail. This
meant fuser never found unix sockets.

pstree finds orphans

Commit 26f9b6c1553d021c0bf9dd85f0647dc6e210948d while fixing the missing
root pid problem introduced another problem where PIDs are out of order.
A process that has a lower PID than its PPID would appear twice with
the tree breaking in strange ways.

We now scan the entire process tree first then perform two checks
 if PID==1 is missing add it
 if a process has no parent, then make it a child of PID==1, either the
 real one or the faked one made in step 1

As this is done after all procsses are found, we don't get the previous
breakage.

Ref: https://sourceforge.net/tracker/?func=detail&aid=3537508&group_id=15273&atid=115273

Signed-off-by: Craig Small <csmall@users.sourceforge.net>

fuser -m regression

While fuser -m /boot worked, fuser -m /dev/blah would show everything.
This was because the filesystem for /dev was added as a target, not the
filesystem /dev/blah.

This simple fix suggested by SF user hanpt works well.

Ref: https://sourceforge.net/tracker/?func=detail&aid=3559232&group_id=15273&atid=115273
Signed-off-by: Craig Small <csmall@users.sourceforge.net>

Merge branch 'master' of ssh://psmisc.git.sourceforge.net/gitroot/psmisc/psmisc

Conflicts:
doc/killall.1
doc/pstree.1

Cleanup man pages

Date uses standard format
Authors section removed
Copyright header in comments

docs: clean up pstree.1

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: clean up prtstat.1

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: clean up peekfd.1

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: clean up killall.1

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: clean up fuser.1

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

misc: fix misspell-check warnings

https://github.com/lyda/misspell-check

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

pstree sorts properly on names

pstree added temporary parents to processes, so if a child process
was lexigraphically before the parents or its "aunties and uncles"
then they were sorted with the parent temporary name "?".

This patch reorders the children in the parents list when they
are renamed.

Start of test scripts for psmisc

Rolled to version 22.19 and fix killall killing all

killall would kill all processes with no arguments, was missing an exit

Update po files with last change

Fixed -w usage SFbug:3528478

Updated readme and po files

Removed remants of pidof

pidof has been in util-linux for a while, but there has been some
remaining code in killall which is now removed.

killall max names fixed

The maximum number of names was not documented and was out by one.
Problem reported by J.A. Bezemer
Bug-Debian: http://bugs.debian.org/677428

killall assert lifetime

Sometimes killall younger-than or older-than would trigger an assert
failure because the process age was too small and was close to zero.

Bug reported and analysed by Frédéric Brière <fbriere@fbriere.net>
Bug-Debian: http://bugs.debian.org/628617

Another sed from from same source

sed uses proper BRE

Apparently even though [0-9]+ works most of the time, its not proper BRE
and doesn't work ALL of the time. Replacing this with [0-9][0-9]* makes
everyone happy.

Noticed by Rich from the busybox project and reported by John Spencer
of the same project.

Conditional install fuser.1

If fuser binary is not installed, don't install fuser man page either

Added AC_CANONICAL_TARGET to configure.ac

This line gives you target_os which is needed for the hurd
dependency lines.

Bug-Debian: http://bugs.debian.org/673485

Update configure.ac to 22.17

Final 22.17 release changes

Updated language files

My thanks again to all the translators for the translation project who
work on these translations.

Made rc1 ready for the translators

pstree displays PGIDs

Patch based upon a patch submitted by Alan Grow.
The -g flag shows program group IDs, much like the PIDs display.

Bug-Sourceforge:
http://sourceforge.net/tracker/?func=detail&aid=3471056&group_id=15273&atid=315273

3 fixes for Cygwin

The attached patch fixes three issues of fuser on Cygwin.:

- Cygwin does not support /proc/net/unix. The call to fill_unix_cache
  always generates an error message that /proc/net/unix couldn't be
  found, so the patch disables the call for Cygwin. An alternative
  fix would be to avoid the error message entirely.

- `ls /dev' on Cygwin only lists manually added entries to /dev, not the
  default files suported by Cygwin internally. Also,
  /proc/$PID/mountinfo doesn't exist up to Cygwin 1.7.11. This
  breaks fuser when using the _LISTS_H option.

- If you don't include lists.h, the code in check_dir doesn't work well.
  st.st_ino is never set to 0, so the timeout(stat, ...) is never
  called. Also, thedev never gets a valid value in this case, but it's
  used subsequently for comparisons.

Fake pstree root with kernels with hidepid

Kernels that hide pids hide pid=1 which pstree needs to show any
output.  If pstree detects this it now makes a "fake" root.

Bug-Sourceforge: https://sourceforge.net/tracker/?func=detail&aid=3509672&group_id=15273&atid=115273
Signed-off-by: Craig Small <csmall@enc.com.au>

fuser compiles with PIE

Some people have reported that fuser, specifically src/lists won't
compile with PIE enabled. This tiny patch fixes it.

Bug-Gentoo: http://bugs.gentoo.org/394443
Bug-Gentoo: http://bugs.gentoo.org/401851
Bug-Sourceforge: https://sourceforge.net/tracker/?func=detail&aid=3504801&group_id=15273&atid=115273

Updated PO files ready for dist

Merge branch 'master' of ssh://psmisc.git.sourceforge.net/gitroot/psmisc/psmisc

Conflicts:
ChangeLog
src/fuser.c

Remove fuser out of target for hurd-i386

hurd-i386 does not have PATH_MAX which means fuser won't compile.
However the archtecture cannot use fuser anyway so we now don't
even try to compile it for hurd-i386

Bug-Debian: http://bugs.debian.org/673485
Signed-off-by: Craig Small <csmall@enc.com.au>

Use --enable-timeout-stat as well as --enable-timeout-stat=static
for a static background process which does the final stat system calls

Signed-off-by: Werner Fink <werner@suse.de>

Add timeout.c/timeout.h for static background process for doing the stat system calls

Signed-off-by: Werner Fink <werner@suse.de>

Really add ASCII null at command string in add_proc() of pstree.c

Signed-off-by: Werner Fink <werner@suse.de>

Updated changelog to 22.17

fuser: Use correct WITH_TIMEOUT_STAT to conditionalize timeout()

Signed-off-by: Guillem Jover <guillem@hadrons.org>

po file refresh

Null terminate some strings

killall: fix potential string overflow [cppcheck]

[src/killall.c:138]: (warning) scanf without field width limits can crash with huge input data

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Better comm_len fix

The previous fix for comm_len broke checks for process exactly the right
length, this one fixes the original problem and doesn't break that
situation.

Added a variable to the conditional STAT code that was previously
outside the #ifdef and gave an unsed variable warning.

Added hardened CFLAGS

Debian and Ubuntu (at the least) have some CFLAGS that harden the binary
for some typical attacks. psmisc now will have these flags on by default
but checks the compilier supports them.

Set COMM_LEN to 18 to include brackets in name

Also use strncpy in case the command names grow.

Updated configure.ac to 22.15

po file update due to distcheck

Updated the language files

Merge branch 'master' of ssh://psmisc.git.sourceforge.net/gitroot/psmisc/psmisc

Conflicts:
ChangeLog

killall: check fscanf return value

killall.c:138:10: warning: ignoring return value of 'fscanf', declared with attribute warn_unused_result [-Wunused-result]

Signed-off-by: Sami Kerola <sami.kerola@tomtom.com>

prtstat: check fgets return value

prtstat.c:234:8: warning: ignoring return value of 'fgets', declared with attribute warn_unused_result [-Wunused-result]

Signed-off-by: Sami Kerola <sami.kerola@tomtom.com>

Moved sigalarm function in fuser

The sigalarm function was outside an #ifdef which meant it was defined
but not used with certain configure parameters.

Thanks to sami Kerola for pointing out the problem.

build-sys: fix AC_TRY_LINK warnings

configure.ac:71: warning: The macro `AC_TRY_LINK' is obsolete.
configure.ac:71: You should run autoupdate.
../../lib/autoconf/general.m4:2688: AC_TRY_LINK is expanded from...
../../lib/m4sugar/m4sh.m4:606: AS_IF is expanded from...
../../lib/autoconf/general.m4:2032: AC_CACHE_VAL is expanded from...
../../lib/autoconf/general.m4:2053: AC_CACHE_CHECK is expanded from...
m4/gettext.m4:367: gt_INTL_MACOSX is expanded from...
m4/gettext.m4:57: AM_GNU_GETTEXT is expanded from...
configure.ac:71: the top level
configure.ac:71: warning: The macro `AC_TRY_LINK' is obsolete.
configure.ac:71: You should run autoupdate.
../../lib/autoconf/general.m4:2688: AC_TRY_LINK is expanded from...
../../lib/m4sugar/m4sh.m4:606: AS_IF is expanded from...
../../lib/autoconf/general.m4:2032: AC_CACHE_VAL is expanded from...
../../lib/autoconf/general.m4:2053: AC_CACHE_CHECK is expanded from...
m4/gettext.m4:57: AM_GNU_GETTEXT is expanded from...
configure.ac:71: the top level
configure.ac:71: warning: The macro `AC_TRY_LINK' is obsolete.
configure.ac:71: You should run autoupdate.
../../lib/autoconf/general.m4:2688: AC_TRY_LINK is expanded from...
../../lib/m4sugar/m4sh.m4:606: AS_IF is expanded from...
../../lib/autoconf/general.m4:2032: AC_CACHE_VAL is expanded from...
../../lib/autoconf/general.m4:2053: AC_CACHE_CHECK is expanded from...

Signed-off-by: Sami Kerola <sami.kerola@tomtom.com>