发现一个定义变量次序出现的问题

0100

2. #include <unistd.h>
3. #include <stdio.h>
4. #include <sys/types.h>
5. #include <sys/sem.h>
6. #include <sys/ipc.h>
7. #define NUM 5
8. #define TIMES 2
9. union semun {
10. int val;
11. struct semid_ds *buf;
12. ushort *array;
13. };
14. P(int sem,int x,int y)
15. {
16. int num1,num2,i=TIMES;
17. struct sembuf lacquire={sem,-1,SEM_UNDO};
18. struct sembuf racquire={sem,-1,SEM_UNDO};
19. struct sembuf lrelease={sem,1,SEM_UNDO};
20. struct sembuf rrelease={sem,1,SEM_UNDO};
21. lacquire.sem_num=(x+4)%5;
22. racquire.sem_num=y%5;
23. lrelease.sem_num=(x+4)%5;
24. rrelease.sem_num=y%5;
25. for(i=0;i<NUM;i++)
26. {
27.         if((num1=semctl(sem,x,GETVAL),0)==-1)
28.         {perror("semctl");exit(3);}
29.         if((num2=semctl(sem,y,GETVAL),0)==-1)
30.         {perror("semctl");exit(3);}
31.         if(num1&&num2)
32.         {
33.         printf("The No.%d philosopher is eating!\n",x);
34.         if(semop(sem,&racquire,1)==-1)
35.                 {perror("semop");exit(5);}
36.         if(semop(sem,&lacquire,1)==-1)
37.                 {perror("semop");exit(5);}
38.         sleep(1);
39.         if(semop(sem,&lrelease,1)==-1)
40.                 {perror("semop");exit(6);}
41.         if(semop(sem,&rrelease,1)==-1)
42.                 {perror("semop");exit(6);}
43.         }
44.         else
45.         {
46.         printf("The No.%d philosopher is thinking!\n",x);
47.         continue;
48.         }
49. }
50. }
51. main()
52. {
53. key_t ipc_key;
54. struct  semid_ds sem_buf;
55. int sem,i;
56. union semun arg;
57. int *p,*pid,cpid;[color=red]
58. //这里要是改成int *p,*pid,cpid;union semun arg;就则出现：段错误
59. //以前看过一个堆栈生长的贴子，但还不明白当变量是指针时，要压入什么到栈里
60. [/color]ushort sem_array[5]={1,1,1,1,1};
61. *pid=getpid();//get the pid of the parent prodecure
62. p=pid;
63. arg.array=sem_array;
64. ipc_key=ftok(".",'c');
65. if((sem=semget(ipc_key,NUM,IPC_CREAT))==-1)
66. {perror("semget");exit(1);}
67. if(semctl(sem,NUM,SETALL,arg)==-1)
68. {perror("semctl SETALL");exit(2);}
69. /*for(i=0;i<NUM;i++)
70.         {
71.         if((cpid=semctl(sem,i,GETVAL,0))==-1)
72.         {perror("semctl GETVAL");exit(8);}
73.         printf("%5d",cpid);
74.         } */
75. for(i=1;i<NUM+1;i++)
76.         {
77.         if((cpid=fork())==-1)
78.                 {
79.                 perror("fork error!");
80.                 exit(1);
81.                 }
82.         else
83.         if(cpid==0)
84.         {
85.         *(pid+i)=getpid();
86.         break;
87.         }
88.         else
89.         continue;
90.         }
91. for(i=1;i<NUM+1;i++)
92.         {
93.         cpid=getpid();
94.         if(cpid==*(p+i))
95.         {
96.         P(sem,i,i+1);
97.         }
98.         else
99.         continue;
100.         }
101. }

复制代码

无双

没有细看

可能你的指针访问非法了吧

使用调试器跟一下

0100

因为编译时就出现错误。没有生成.o文件。所以无法调试。

无双

我在dev-c++下与
main()
{
union semun {
int val;
struct semid_ds *buf;
unsigned short *array;
};
int *p,*pid,cpid;
union semun arg;
}

这样没错

我想你会不会是其它地方的错误

麥頭

调换前用gcc可以顺利编译和运行
调换后用gcc加-g后直接编译成a.out通过，但运行出现segment failt错误
-g gdb调试显示错误大概在ipc_key=ftok(".",'c');

不懂。关注

麥頭

和顺序无关
搞了半天，
你程序的指针p和pid没有分配空间就用p+1,pid+1的，内存乱了，就segment failt
我已修改：错误处加长线标出

2. #include <unistd.h>
3. #include <stdio.h>
4. #include <sys/types.h>
5. #include <sys/sem.h>
6. #include <sys/ipc.h>
7. #define NUM 5
8. #define TIMES 2
9. union semun {
10.         int val;
11.         struct semid_ds *buf;
12.         ushort *array;
13. };
15. void P(int sem,int x,int y)
16. {
17.         int num1,num2,i=TIMES;
18.         struct sembuf lacquire={sem,-1,SEM_UNDO};
19.         struct sembuf racquire={sem,-1,SEM_UNDO};
20.         struct sembuf lrelease={sem,1,SEM_UNDO};
21.         struct sembuf rrelease={sem,1,SEM_UNDO};
22.         lacquire.sem_num=(x+4)%5;
23.         racquire.sem_num=y%5;
24.         lrelease.sem_num=(x+4)%5;
25.         rrelease.sem_num=y%5;
26.         for(i=0;i<NUM;i++)
27.         {
28.                        if((num1=semctl(sem,x,GETVAL),0)==-1)
29.                                {perror("semctl");exit(3);}
30.                        if((num2=semctl(sem,y,GETVAL),0)==-1)
31.                                {perror("semctl");exit(3);}
32.                        if(num1&&num2){
33.                                printf("The No.%d philosopher is eating!\n",x);
34.                                if(semop(sem,&racquire,1)==-1)
35.                                        {perror("semop");exit(5);}
36.                                if(semop(sem,&lacquire,1)==-1)
37.                                        {perror("semop");exit(5);}
38.                                sleep(1);
39.                                if(semop(sem,&lrelease,1)==-1)
40.                                        {perror("semop");exit(6);}
41.                                if(semop(sem,&rrelease,1)==-1)
42.                                        {perror("semop");exit(6);}
43.                        }
44.                        else{
45.                        printf("The No.%d philosopher is thinking!\n",x);
46.                        continue;
47.                    }
48.         }
49. }
51. //
52. int main()
53. {
54.         key_t ipc_key;
55. //        struct  semid_ds sem_buf;___________________用-Wall参数编译可看出这个变量程序从没用过
56.         int sem,i;
57.         ushort sem_array[5]={1,1,1,1,1};
58.        
59.        
60.        
61.        
62.         //union semun arg;
63.         int *p,*pid,cpid;
64.         union semun arg;
65. //分配空间………………………………………………………………
66.         pid=(int *)(malloc(5*sizeof(int)));
67.         p=(int *)(malloc(5*sizeof(int)));
68.        
69.        
70.        
71. //这里要是改成int *p,*pid,cpid;union semun arg;就则出现：段错误
72. //以前看过一个堆栈生长的贴子，但还不明白当变量是指针时，要压入什么到栈里
73.        
74.         *pid=getpid();//get the pid of the parent prodecure
75.         p=pid;
76.         arg.array=sem_array;
77.         ipc_key=ftok(".",'c');
78.         if((sem=semget(ipc_key,NUM,IPC_CREAT))==-1)
79.                 {perror("semget");exit(1);}
80.         if(semctl(sem,NUM,SETALL,arg)==-1)
81.                 {perror("semctl SETALL");exit(2);}
82. //        /*for(i=0;i<NUM;i++)
83. //                {
84. //      if((cpid=semctl(sem,i,GETVAL,0))==-1)
85. //        {perror("semctl GETVAL");exit(8);}
86. //        printf("%5d",cpid);
87. //        } */
90.         for(i=1;i<NUM+1;i++){
91.                 if((cpid=fork())==-1){
92.                         perror("fork error!");
93.                         exit(1);
94.                 }
95. //---------------------------------------------------pid+1//以前是指针pid+1没有分配空间
96.                 else if(cpid==0){
97.                         *(pid+i)=getpid();
98.                         break;
99.                 }
100.                 else
101.                 continue;
102.         }
103.        
104.         for(i=1;i<NUM+1;i++){
105.                 cpid=getpid();
106.   //……………………………………………………………………错误同上p
107.                 if(cpid==*(p+i)){
108.                         P(sem,i,i+1);
109.                 }
110.                 else
111.                 continue;
112.         }
113.         exit(0);
115. }

复制代码

指针要小心用
附带问这里p和pid要free吗？

kj501

当然，动态分配的内存都要自己free。