APUE2里15－2程序异常出错，高手帮看看

wawxdyy

下面这个代码是我照着APUE2的第十五章的15-2例子写的，可是在执行的时候直接就死掉了，我用echo $?得到141，这个按我的理解应该是141－128＝13，也就是权限出错吧，可是我查看了一下，权限没问题，请大家帮我调试一下，看哪里出的问题，谢谢

1. 
   
2. #include <sys/wait.h>
   
3. #include <stdio.h>
   
4. #include <stdlib.h>
   
5. #include <string.h>
   
6. #include <unistd.h>
   
7. #include <errno.h>
   
8. 
   
9. #define        DEF_PAGER        "/bin/more"
   
10. #define        MAX_LINE        2048
    
11. 
    
12. int
    
13. main(int argc, char *argv[])
    
14. {
    
15.         int        n;
    
16.         int        fd[2];
    
17.         pid_t        pid;
    
18.         char        *pager, *argv0;
    
19.         char        line[MAX_LINE];
    
20.         FILE        *fp;
    
21.        
    
22.         if (argc != 2) {
    
23.                 printf("usage: %s <filename>\n", argv[0]);
    
24.                 exit(1);
    
25.         }
    
26.        
    
27.         if ((fp = fopen(argv[1], "r")) == NULL) {
    
28.                 printf("open %s failed: %s\n", argv[1], strerror(errno));
    
29.                 exit(1);
    
30.         }
    
31.        
    
32.         if (pipe(fd) < 0) {
    
33.                 printf("create pipe failed: %s\n", strerror(errno));
    
34.                 exit(1);
    
35.         }
    
36.        
    
37.         if ((pid = fork()) < 0) {
    
38.                 printf("%d fork failed: %s\n", getpid(), strerror(errno));
    
39.                 exit(1);
    
40.         } else if (pid > 0) {
    
41.                 close(fd[0]);
    
42.                 while (fgets(line, MAX_LINE, fp) != NULL) {
    
43.                         n = strlen(line);
    
44.                         if (write(fd[1], line, n) != n) {
    
45.                                 printf("write failed: %s\n", strerror(errno));
    
46.                                 exit(1);
    
47.                         }
    
48.                 }
    
49.                
    
50.                 if (ferror(fp)) {
    
51.                         printf("read %s failed: %s\n", argv[1], strerror(errno));
    
52.                         exit(1);
    
53.                 }
    
54.                
    
55.                 close(fd[1]);
    
56. 
    
57.                 if ((waitpid(pid, NULL, 0)) < 0) {
    
58.                         printf("wait %d failed: %s\n", pid, strerror(errno));
    
59.                         exit(1);
    
60.                 }
    
61.                
    
62.                 exit(0);
    
63.         } else {
    
64.                 close(fd[1]);
    
65.                
    
66.                 if (fd[0] != STDIN_FILENO) {
    
67.                         if ((dup2(fd[0], STDIN_FILENO)) != STDIN_FILENO) {
    
68.                                 printf("dup2 failed: %s\n", strerror(errno));
    
69.                                 exit(1);
    
70.                         }
    
71.                         close(fd[0]);
    
72.                 }
    
73.                
    
74.                 if ((pager = getenv("PAGER")) != NULL) {
    
75.                         pager = DEF_PAGER;
    
76.                 }
    
77.                
    
78.                 if ((argv0 = strrchr(pager, '/')) == NULL)
    
79.                         argv0 = pager;
    
80.                 else
    
81.                         argv0++;
    
82.                
    
83.                 if ((execl(pager, argv0, (char *)0)) < 0) {
    
84.                         printf("execl %s failed: %s\n", pager, strerror(errno));
    
85.                         exit(1);
    
86.                 }
    
87.         }
    
88.         exit(0);
    
89. }
    
90. 
    

复制代码

wawxdyy

下面是我用gdb ./a.out a调试的结果（a的内容是"aaa"三个字符）

1. 
   
2. GNU gdb 6.4.90-debian
   
3. Copyright (C) 2006 Free Software Foundation, Inc.
   
4. GDB is free software, covered by the GNU General Public License, and you are
   
5. welcome to change it and/or distribute copies of it under certain conditions.
   
6. Type "show copying" to see the conditions.
   
7. There is absolutely no warranty for GDB.  Type "show warranty" for details.
   
8. This GDB was configured as "i486-linux-gnu"...Using host libthread_db library "/lib/tls/i686/cmov/libthread_db.so.1".
   
9. 
   
10. "/home/ice/work/source/15/a" is not a core dump: 不可识别的文件格式
    
11. (gdb) r
    
12. Starting program: /home/ice/work/source/15/a.out
    
13. Failed to read a valid object file image from memory.
    
14. usage: /home/ice/work/source/15/a.out <filename>
    
15. 
    
16. Program exited with code 01.
    

复制代码

wawxdyy

解决了，不好意思，自己太粗心了

1. 
   
2. if ((pager = getenv("PAGER")) != NULL) {
   
3.                          pager = DEF_PAGER;
   
4.                  }
   

复制代码

改为

1. 
   
2. if ((pager = getenv("PAGER")) == NULL) {
   
3.                          pager = DEF_PAGER;
   
4.                  }
   

复制代码

paul1983

...........
看来粗心是大多数人的权利啊，真幸福，我也有这个权利