又发现APUE第二版(中文版)的一个BUG

又发现APUE第二版(中文版)的一个BUG

2008-12-13 23:25 795人阅读 评论(2) 收藏 举报

cmdlinux编程终端nullfp

APUE是学习Linux编程最权威的一本书，但权威也不是绝对的。上周读到线程一章，在第304页发现一个BUG：
   函数foo_alloc(void)中:
         fp->f_next = h[idx];
         fh[idx] = fp->f_next;   改为  ==> fh[idx] = fp;
                       ^^^^^^^^^^^^^                       
      今天调试第344页的daemonize函数时又发现一个BUG,在damonize的最后加入 sigsuspend调用，再添加main函数让进程跑起来，编译后发现进程运行后立即退出，达不到damon进程的要求。 分析代码，发现是setsid()的位置不对。
      本书219页阐述了 setsid的三个作用：
               1＞ 该进程变成新会话首进程。
               2＞ 该进程成为一个新进程组的组长进程。
               3＞ 该进程没有控制终端。
      代码中调用setsid创建了一个只有一个进程的进程组，按书的代码，setsid之前，父进程已退出，那么setsid之后，子进程所在的进程组变成孤儿进程组，POSIX.1要求向新的孤儿进程组中处于停止状态的每一个进程发送挂断信号(SIGHUP)，而系统对挂断信号的系统默认动作是终止该进程，所以在调用sigsuspend之后，进程接收SIGHUP信号便退出。
      
 我改写了书中的代码(程序清单13-1)：

1. #include "apue.h"
2. #include <syslog.h>
3. #include <fcntl.h>
4. #include <signal.h>
5. #include <sys/resource.h>
7. void daemonize(const char *cmd)
8. {
9.     int                 i, fd0, fd1, fd2;
10.     pid_t               pid;
11.     struct rlimit       rl;
12.     struct sigaction    sa;
13.     sigset_t waitmask;
15.     /*
16.      * Clear file creation mask.
17.      */
18.     umask(0);
20.     /*
21.      * Get maximum number of file descriptors.
22.      */
23.     if (getrlimit(RLIMIT_NOFILE, &rl) < 0)
24.         err_quit("%s: can't get file limit", cmd);
26.     /*
27.      * Become a session leader to lose controlling TTY.
28.      */
29.     if ((pid = fork()) < 0)
30.         err_quit("%s: can't fork", cmd);
31.     else if (pid != 0) /* parent */
32.         exit(0);
37.     /*
38.      * Ensure future opens won't allocate controlling TTYs.
39.      */
40.     sa.sa_handler = SIG_IGN;
41.     sigemptyset(&sa.sa_mask);
42.     sa.sa_flags = 0;
43.     if (sigaction(SIGHUP, &sa, NULL) < 0)
44.         err_quit("%s: can't ignore SIGHUP");
45.     if ((pid = fork()) < 0)
46.         err_quit("%s: can't fork", cmd);
47.     else if (pid != 0) /* parent */
48.         exit(0);
50.     setsid();  /*setsid在SIGHUP被忽略后执行，避免因系统对孤儿进程发送SIGHUP信号而导致进程退出。*/
53.     /*
54.      * Change the current working directory to the root so
55.      * we won't prevent file systems from being unmounted.
56.      */
57.     if (chdir("/") < 0)
58.         err_quit("%s: can't change directory to /");
60.     /*
61.      * Close all open file descriptors.
62.      */
63.     if (rl.rlim_max == RLIM_INFINITY)
64.         rl.rlim_max = 1024;
65.     for (i = 0; i < rl.rlim_max; i++)
66.         close(i);
68.     /*
69.      * Attach file descriptors 0, 1, and 2 to /dev/null.
70.      */
71.     fd0 = open("/dev/null", O_RDWR);
72.     fd1 = dup(0);
73.     fd2 = dup(0);
75.     /*
76.      * Initialize the log file.
77.      */
78.     openlog(cmd, LOG_CONS, LOG_DAEMON);
79.     if (fd0 != 0 || fd1 != 1 || fd2 != 2) {
80.         syslog(LOG_ERR, "unexpected file descriptors %d %d %d",
81.           fd0, fd1, fd2);
82.         exit(1);
83.     }
85.     /*我添加了以下三行*/
86.     sigemptyset(&waitmask);
87.     if (sigsuspend(&waitmask) != -1)
88.         err_sys("sigsuspend error");
89. }
91. /*还要添加一个main函数让代码跑起来。*/
92. int main() {
93.     daemonize("hellor,world");   
94.     return 0;
95. }