[Nagiosplug-devel] [ nagiosplug-Patches-1455619 ] check_procs segfaults with zombies

[SourceForge.net]

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