Re: [Comm] Re: /sbin/ifup ppp0 - некорректная работа по Ctrl-C

[Valentin Lavrinenko <lvu@uaservice.com.ua>]

[-- Attachment #1: Type: text/plain, Size: 1081 bytes --]

Hi!

Michael Shigorin wrote:

>Вы ещё не списались с Денисом?
>
Еще нет.

В общем, я, похоже, нашел, в чем проблема, и даже сумел ее поправить.
При этом наткнулся на баг (точнее, даже пару связанных) в pppd, повесил
на http://ppp.samba.org/cgi-bin/ppp-bugs/incoming?id=1285.

К сожалению, я не могу зарегистрироваться в багзилле - видимо, наш сервак
рубит письмо с подтверждением. Так что патч приаттачен к письму. Основная
идея: когда юзер нажал Ctrl-C, pppp-watch смотрит .pid-файл, создаваемый 
pppd,
и, если нашел - по полученному pid'у убивает pppd. Однако, pppd создает этот
файл не сразу, а только когда установится соединение (отработает 
chat-скрипт,
например). Поэтому, если .pid-файл не найден, то мы ищем pppd, запущенный
с нужным на параметром, в /proc. Если нашли - ждем 10 секунд (должно 
хватить),
периодически проверяя, не появился ли .pid-файл, как только появился - 
убиваем
pppd.

Можно юыло бы, найдя pppd в /proc, сразу убивать pppd, но этому мешает 
упомянутая
выше бага.

Если этот патч появится в Сизифе - буду очень рад, если в backports - 
еще больше :)

[-- Attachment #2: net-scripts-pppd_sigterm.patch --]
[-- Type: text/x-patch, Size: 2344 bytes --]

diff -ur net-scripts-0.4.9.1/src/ppp-watch.c net-scripts-0.4.9.1-lvu/src/ppp-watch.c
--- net-scripts-0.4.9.1/src/ppp-watch.c	2001-03-05 02:22:06 +0200
+++ net-scripts-0.4.9.1-lvu/src/ppp-watch.c	2005-10-21 10:31:23 +0300
@@ -75,6 +75,7 @@
 #include <sys/wait.h>
 #include <termios.h>
 #include <net/if.h>
+#include <glob.h>
 
 #include "shvar.h"
 #include "alloc.h"
@@ -116,6 +117,43 @@
 }
 
 
+static int
+get_pppd_pid(char *device) {
+    int i;
+    glob_t globbuf;
+    int ge = glob("/proc/*/cmdline", 0, NULL, &globbuf);
+    if (!ge) {
+	for (i = 0; i < globbuf.gl_pathc; ++i) {	    
+	    FILE *fcmdline;
+	    char cmdline[1024]; // hope this will be enough
+	    if (fcmdline = fopen(globbuf.gl_pathv[i], "r")) {
+		int num_read;
+		num_read = fread(cmdline, 1, 1024, fcmdline);
+		if (num_read > 0 && !strcmp(cmdline, "/usr/sbin/pppd")) {
+		    int prev_linkname = 0;
+		    char *p = cmdline;
+		    while (*p && p - cmdline < 1024) {
+			if (prev_linkname && !strcmp(p, device)) {
+			    int retval;
+			    globbuf.gl_pathv[i][strlen(globbuf.gl_pathv[i])-8] = 0;
+			    retval = atoi(globbuf.gl_pathv[i]+6);
+			    globfree(&globbuf);
+			    fclose(fcmdline);
+			    return retval;
+			}
+			prev_linkname = !strcmp(p, "linkname");
+			p += strlen(p) + 1;
+		    }		    
+		}
+		fclose(fcmdline);
+	    }
+	}
+	globfree(&globbuf);
+	return 0;
+    }
+    else
+	return -1;
+}
 
 static void
 detach(int now, int parentExitCode, char *device) {
@@ -550,6 +588,20 @@
 	    if (physicalDevice) { free(physicalDevice); physicalDevice = NULL; }
 	    physicalDevice = pppLogicalToPhysical(&pppdPid, real_device);
 	    if (physicalDevice) { free(physicalDevice); physicalDevice = NULL; }
+	    if (!pppdPid && get_pppd_pid(device))
+	    {
+		// We suspect that pppd is running, but haven't created
+		// .pid file yet. So we'll wait for 10 seconds, checking
+		// for .pid file every 10 milliseconds.
+		int i;
+		for (i = 0; i < 1000 && !pppdPid && get_pppd_pid(device) !=0 ; ++i)
+		{
+		    usleep(10000);
+		    physicalDevice = pppLogicalToPhysical(&pppdPid, real_device);
+		    if (physicalDevice) { free(physicalDevice); physicalDevice = NULL; }
+		}
+		sleep(2); // Else pppd may hangup, seems there's a bug in it
+	    }
 	    if (!pppdPid) cleanExit(35);
 	    kill(pppdPid, sendsig);
 	    if (sendsig == SIGKILL) {