[Nagiosplug-devel] [ nagiosplug-Patches-1455619 ] check_procs segfaults with zombies
SourceForge.net
noreply at sourceforge.net
Wed Apr 5 19:21:05 CEST 2006
Patches item #1455619, was opened at 03/21/06 12:03
Message generated for change (Comment added) made by sf-robot
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=1455619&group_id=29880
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Bugfix
Group: None
>Status: Closed
Resolution: None
Priority: 5
Submitted By: gerhard lausser (lausser)
Assigned to: Ton Voon (tonvoon)
Summary: check_procs segfaults with zombies
Initial Comment:
Hi,
under solaris, ps produces output like this for
zombie processes:
Z 55618 28898 15263 0 0 0.0
The statement
cols = sscanf (input_line, PS_FORMAT, PS_VARLIST);
which is actually this one:
...sscanf (input_line, "%s %d %d %d %d %d %f %s%n",
procstat,&procuid,&procpid,&procpid,&procvsz,&procrss,
&procpcpu,procprog,&pos);
does not fill the pos variable with a value because
there is no match for the last %s. pos keeps the
value it got when the last time a non-zombie process
was found.
pos points to the arguments following procprog which
is senseless in this case anyway. but later it is
used to initialize the procargs array.
asprintf (&procargs, "%s", input_line + pos);
In the case of a zombie pos has a value related to
the last input_line but not the current one, so an
illegal byte beyond the strings end may be accessed.
I made a patch which takes this into account.
Greetings from Munich,
Gerhard
----------------------------------------------------------------------
>Comment By: SourceForge Robot (sf-robot)
Date: 04/05/06 19:20
Message:
Logged In: YES
user_id=1312539
This Tracker item was closed automatically by the system. It was
previously set to a Pending status, and the original submitter
did not respond within 14 days (the time period specified by
the administrator of this Tracker).
----------------------------------------------------------------------
Comment By: Ton Voon (tonvoon)
Date: 03/22/06 05:15
Message:
Logged In: YES
user_id=664364
Gerhard,
Thanks for the patch. For the upcoming 1.4.3 release, Solaris processes will
be read differently. A new executable, pst, will be created which will return
the equivalent ps output. Please can you try this and see if it fixes your
zombie issues.
If not, I'll be happy to accept a patch against that.
Please try the snapshot at http://nagiosplug.sourceforge.net/snapshot/
Ton
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=1455619&group_id=29880
More information about the Devel
mailing list