https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c1 --- Comment #1 from Freek de Kruijf 2011-10-13 10:38:33 UTC --- *** Bug 721655 has been marked as a duplicate of this bug. *** http://bugzilla.novell.com/show_bug.cgi?id=721655 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c3 --- Comment #3 from Freek de Kruijf 2011-10-13 20:24:54 UTC --- I retried it with the full name and it still returns the wrong values. # ps aux | grep amavis root 4553 0.0 0.0 6788 872 pts/1 S+ 22:19 0:00 grep amavis # rcamavis start Job failed. See system logs and 'systemctl status' for details. # ps aux | grep amavis vscan 4570 22.2 9.5 217152 97124 ? Ss 22:20 0:01 /usr/sbin/amavisd (master) vscan 4574 0.0 9.4 218424 96156 ? S 22:20 0:00 /usr/sbin/amavisd (virgin child) vscan 4575 0.0 9.4 218424 96148 ? S 22:20 0:00 /usr/sbin/amavisd (virgin child) root 4577 0.0 0.0 6788 872 pts/1 S+ 22:20 0:00 grep amavis # checkproc /usr/sbin/amavisd ; echo $? 3 # checkproc -p /var/spool/amavis/amavisd.pid /usr/sbin/amavisd ; echo $? 1 # -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c5 Freek de Kruijf changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW InfoProvider|f.de.kruijf@gmail.com | --- Comment #5 from Freek de Kruijf 2011-10-14 14:10:51 UTC --- Obviously I am user root, see the # in front of the commands. No. I started apache2 using 'rcapache2 start'. Then used # checkproc /usr/sbin/httpd2 ; echo $? and the result was 0. As I suggested earlier, the cause could be that /usr/sbin/amavisd is a perl script. You can easily reproduce it. Only two lines in /etc/amavisd.conf may need some change. $mydomain = ... and $myhostname = .... needs to be added if uname -n does not provide a fully qualified name. After that you start amavisd by giving 'rcamavis start'. It reports an error because checkproc returns the wrong value, but is is running. After that you give the checkproc command, which gives you the wrong answers. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c7 --- Comment #7 from Freek de Kruijf 2011-10-14 15:27:43 UTC --- OK. This means that /etc/init.d/amavis needs a rewrite, because it relies on checkproc. Apparently this is a deviation from earlier versions of checkproc. Because /etc/init.d/amavis in openSUSE 12.1 is the same as in openSUSE 11.4. Amavis worked in openSUSE 11.4. Can you suggest how to check for running amavisd processes? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c9 --- Comment #9 from Freek de Kruijf 2011-10-14 16:43:37 UTC --- The problem is probably a combination of checkproc not supposed to work for perl scripts but it did. In openSUSE 11.4 checkproc worked OK for detecting whether amavisd was running or not. Now in openSUSE 12.1 it does not work that way. I will try to run the version of amavisd-new from openSUSE 11.4 in openSUSE 12.1 and see what happens. After all it is just a perl script. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c11 --- Comment #11 from Dr. Werner Fink 2011-10-17 07:35:41 UTC --- the /proc file system or better the permssions have changed betwen the kernel used on 11.4 and the kernel used for 21.1 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c13 --- Comment #13 from Dr. Werner Fink 2011-10-18 09:36:39 UTC --- Just submitted a version of sysvinit which includes killproc/checkproc/startproc/start_daemon knowing an option -x for detecting scripts overwriting their command line with the help of the upto 15 bytes long filename found in /proc/<pid>/stat If accepted to 12.1 the boot script for amavis could be modified to use the option -x for checkproc and all other tools mentioned above. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c15 --- Comment #15 from Freek de Kruijf 2011-10-20 07:47:37 UTC --- Unfortunately the bugfix does not work: eik121b1v:~ # rcamavis start Job failed. See system logs and 'systemctl status' for details. eik121b1v:~ # ps aux | grep amavisd vscan 3608 9.5 9.6 218908 98880 ? Ss 09:34 0:01 /usr/sbin/amavisd (master) vscan 3612 0.0 9.5 220180 97916 ? S 09:34 0:00 /usr/sbin/amavisd (virgin child) vscan 3613 0.0 9.5 220180 97912 ? S 09:34 0:00 /usr/sbin/amavisd (virgin child) root 3617 0.0 0.0 6792 872 pts/1 S+ 09:34 0:00 grep amavisd eik121b1v:~ # checkproc -v amavisd eik121b1v:~ # checkproc -v /usr/sbin/amavisd eik121b1v:~ # checkproc -x amavisd ; echo $? 3 eik121b1v:~ # checkproc -x -p /var/spool/amavis/amavisd.pid amavisd ; echo $? 1 eik121b1v:~ # checkproc -x /usr/sbin/amavisd ; echo $? 3 eik121b1v:~ # checkproc -x -p /var/spool/amavis/amavisd.pid /usr/sbin/amavisd ; echo $? 1 eik121b1v:~ # more /proc/$(cat /var/spool/amavis/amavisd.pid)/stat 3608 (/usr/sbin/amavi) S 1 3608 3608 0 -1 4202560 11045 0 14 0 127 8 0 0 20 0 1 0 8609 224161792 24720 18446744073709551615 4194304 5645308 140735348404400 140735348403656 140307836922899 0 0 4224 81927 0 0 0 17 0 0 0 20 0 0 eik121b1v:~ # -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c17 Jiri Slaby changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO CC| |jslaby@suse.com InfoProvider| |werner@suse.com --- Comment #17 from Jiri Slaby 2011-10-31 10:00:12 UTC --- (In reply to comment #16)

IMHO this is a kernel bug! The 15 bytes (16 including ASCII zero) should not use the full path but the filename of the program or the script.

It contains whatever the task sets as its comm. Either by prctl(PR_SET_NAME) or by writing to /proc/<pid>/comm file. Maybe amavis sets its full path? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c19 --- Comment #19 from Dr. Werner Fink 2011-11-02 12:23:12 UTC --- @Jiri: without having the real filename of the script is becomes impossible to detect any running script by its filename, tools like pidof (with option -x) and startproc/killproc/checkproc become useless for scripts. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c21 Dr. Werner Fink changed: What |Removed |Added ---------------------------------------------------------------------------- InfoProvider|varkoly@suse.com |mls@suse.com --- Comment #21 from Dr. Werner Fink 2011-11-07 10:27:04 UTC --- (In reply to comment #20) Maybe this is a perl feature if a script sets $0 then use /proc/<pid>/comm of prctl(PR_SET_NAME), now let us aks a perl wizard like our perl maintainer :) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c23 Andreas Bolsch changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andreas.bolsch@alumni.tu-be | |rlin.de --- Comment #23 from Andreas Bolsch 2011-11-19 14:01:41 UTC --- Same problem appears for spamd from spamassassin in 12.1 as it is a Perl script, too ... -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c25 Michael Schröder changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW CC| |mls@suse.com InfoProvider|mls@suse.com | --- Comment #25 from Michael Schröder 2011-11-21 13:47:33 UTC --- To shed some light on this, perl-5.14 uses prctl() to set $0, perl-5.12 did not do this. Here's the perldelta entry: Assignment to C<$0> sets the legacy process name with C on Linux On Linux the legacy process name will be set with L, in addition to altering the POSIX name via C as perl has done since version 4.000. Now system utilities that read the legacy process name such as ps, top and killall will recognize the name you set when assigning to C<$0>. The string you supply will be cut off at 16 bytes, this is a limitation imposed by Linux. amavisd reads $0 (which results in /usr/sbin/amavisd), modifies it, and writes it back to $0. perl then does the prctl() call with the new name. One can argue that perl should just use the basename for the prctl() call, to be somewhat compatible to the kernel's setup_new_exec() call. But I don't think this would fix the bug, as amavisd modifies $0 by appending/prepending stuff, so checkproc probably won't detect a match (but maybe it cuts off after a space?). Seems like there's no trace of the old name left when a prctl(PR_SET_NAME) has been done. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c27 Stephan Kulow changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |coolo@suse.com --- Comment #27 from Stephan Kulow 2011-11-21 15:24:50 CET --- checkproc looks for "(" + base + ")", so there is basically no chance to find it. But the checkproc code is marked as /* High risky */, so my suggestion would be: don't change $0 for the master process in amivsd to fix it. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c29 --- Comment #29 from Dr. Werner Fink 2011-11-21 14:37:53 UTC --- (In reply to comment #27) "(" + base + ")" is used for /proc/<pid>/stat the function checkscripts() is used for /proc/<pid>/cmdline (In reply to comment #28) As I've already written a few years back it is /* High risky */ -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c31 Jeff Mahoney changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO CC| |jeffm@suse.com InfoProvider| |werner@suse.com --- Comment #31 from Jeff Mahoney 2012-03-24 23:52:00 EDT --- So what's the verdict here? I can see in 3.3 that calling /home/jeffm/tmp.pl only displays 'tmp.pl' in stat. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c33 --- Comment #33 from Dr. Werner Fink 2012-03-26 14:16:08 UTC --- Just compared Stephan's script on an old kernel with a kernel from 12.1: old : head -n 1 /proc/27985/status && cat /proc/27985/stat Name: t.pl 27985 (t.pl) S 32327 27985 32327 34841 27985 4202496 541 0 0 0 0 0 0 0 20 0 1 0 475999684 6320128 439 1771696128 134512640 136972940 3218382528 3218381524 4294960176 0 0 128 0 0 0 0 17 0 0 0 0 0 0 12.1 : head -n 1 /proc/7490/status && cat /proc/7490/stat Name: /suse/werner/Te 7490 (/suse/werner/Te) S 7450 7490 7450 34818 7490 4202496 539 0 0 0 0 0 0 0 20 0 1 0 476517410 15196160 371 7091580928 4194304 5647116 140735570242032 40735570240936 140594171436528 0 0 128 0 0 0 0 17 0 0 0 0 0 0 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c35 --- Comment #35 from Jeff Mahoney 2012-03-26 14:30:18 EDT --- It's a userspace issue. I used this script so I don't need to do ps and find the pid: #! /usr/bin/perl -T $0 = "$0 (master)"; open(STAT, "/proc/$$/stat") or die; my $line = <STAT>; print $line; Using a SLES11 SP2 system for both runs: jeffm@sled1:~> uname -r ; /home/jeffm/tmp.pl 3.1.0-1.2-default 4595 (tmp.pl) R 4550 4595 4550 34817 4595 4202496 521 0 0 0 0 0 0 0 20 0 1 0 26560 16584704 420 14329221120 4194304 5805580 140736756691568 140736756689928 140386202527568 0 0 128 0 0 0 0 17 1 0 0 0 0 0 jeffm@sled1:~> uname -r ; /home/jeffm/tmp.pl 3.0.13-default 4373 (tmp.pl) R 4271 4373 4271 34816 4373 4202496 522 0 0 0 0 0 0 0 20 0 1 0 7127 16584704 421 14329221120 4194304 5805580 140735039436048 140735039435640 140725939357520 0 0 128 0 0 0 0 17 0 0 0 0 0 0 Using an openSUSE 12.1 system for both runs: sled2:ß # uname -r ; /home/jeffm/tmp.pl 3.0.13-0.27-default 2946 (/home/jeffm/tmp) R 2114 2946 2114 1088 2946 4202496 507 0 0 0 0 0 0 0 20 0 1 0 4693 15196160 359 18446744073709551615 4194304 5647116 140734911709344 140734851228808 139719650340272 0 0 128 0 0 0 0 17 12 0 0 0 0 0 sled2:ß # uname -r ; /home/jeffm/tmp.pl 3.1.9-1.4-desktop 4814 (/home/jeffm/tmp) R 2541 4814 2541 1088 4814 4202496 523 0 0 0 0 0 0 0 20 0 1 0 11850 16482304 426 18446744073709551615 4194304 5647116 140735712501248 140735712500728 140045854358960 0 0 128 0 0 0 0 17 4 0 0 5 0 0 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c37 Dr. Werner Fink changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO InfoProvider| |jeffm@suse.com --- Comment #37 from Dr. Werner Fink 2012-03-27 07:13:50 UTC --- (In reply to comment #35) It is *not* a user space issue, it is a regression caused by the kernel version update. Before the kernel update no perl script was able to override the string in /proc/<pid>/stat nor the Name in /proc/<pid>/status, now after the kernel version update any perl script can do. This breaks the compatibility from SLES11-SP1 to SLES11-SP2 and this is what I call a regression. And I'd like to know how to determine the name of the used script below /proc/<pid>/ with kernels 3.+ with user space tools in SLES11-SP2 to be able to determine the script name by a system tools like pgrep(1), pkill(1), pidof(1), and my tools like checkproc(8). This kernel version update has take the edge off former working tools. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c39 Michael Schröder changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|mls@suse.com |werner@suse.com --- Comment #39 from Michael Schröder 2012-03-28 09:56:24 UTC --- The perl maintainer already discussed this in comment #25 ;) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c41 --- Comment #41 from Dr. Werner Fink 2012-03-28 10:13:17 UTC --- Beside this there is no 'RESOLVED' with 'NOTSUPPORTEDANYMORE'. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=723708 https://bugzilla.novell.com/show_bug.cgi?id=723708#c43 Dr. Werner Fink changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|werner@suse.com |varkoly@suse.com --- Comment #43 from Dr. Werner Fink 2012-03-28 10:51:21 UTC --- maintainer of amavisd-new is varkoly@suse.com and not me, that is I can not fix this. Then the question rises if a patched amavisd will miss a major feature, that is to have a new command line. The next question is what about all the other perl scripts? IMHO this perl `feature' isn't one -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.