在一个daemon程序中调用system来执行一段脚本,返回值总是-1.脚本简单的不能再简单了,只有一句exit 0,但还是返回-1.
查了很多资料,也没能找出原因,最后经过我反复的调试,终于发现了一个惊天动地的秘密!!
返回-1的原因是因为进程将SIGCHLD屏蔽了!!
确实我的程序在main函数里面就调用signal(SIGCHLD, SIG_IGN);将SIGCHLD屏蔽了!
system的调用过程主要是调三个函数
1.fork()
2.execl()
3.wait()
wait需要依赖SIGCHLD信号,所以屏蔽了SIGCHLD信号就永远得不到system正确的返回值了!
另外附一段正确判断脚本执行的方法:
system的返回值是按不同的字节置位的,0~7位和8~15位置的是不同的内容,判断脚本是否正确执行的正确做法是:
查了很多资料,也没能找出原因,最后经过我反复的调试,终于发现了一个惊天动地的秘密!!
返回-1的原因是因为进程将SIGCHLD屏蔽了!!
确实我的程序在main函数里面就调用signal(SIGCHLD, SIG_IGN);将SIGCHLD屏蔽了!
system的调用过程主要是调三个函数
1.fork()
2.execl()
3.wait()
wait需要依赖SIGCHLD信号,所以屏蔽了SIGCHLD信号就永远得不到system正确的返回值了!
另外附一段正确判断脚本执行的方法:
system的返回值是按不同的字节置位的,0~7位和8~15位置的是不同的内容,判断脚本是否正确执行的正确做法是:
(1)-1 != status
(2)WIFEXITED(status)为真
(3)0 == WEXITSTATUS(status)
下面来解释一下后两个返回值的含义:
最后写了一段小的测试程序。下面 这段代码在2.6内核的系统上总是返回-1,但是奇怪的是在2.4内核的系统上返回0.
#include
using namespace std;
#include
#include
#include
#include
#include
int main()
{
cout<<"pid: "<<getpid()<<endl;
int ret = 0;
signal(SIGCHLD, SIG_IGN);
ret = system("./wyf.sh");
cout<<"ret: "<<ret<<endl;
return 0;
}
原文链接:http://blog.chinaunix.net/uid-12274566-id-3508850.html
下面来解释一下后两个返回值的含义:
1,WIFEXITED(status) 这个宏用来指出子进程是否为正常退出的,如果是,它会返回一个非零值。
2,WEXITSTATUS(status) 当WIFEXITED返回非零值时,我们可以用这个宏来提取子进程的返回值,如果子进程调用exit(5)退出,WEXITSTATUS(status)就会返回5;如果子进程调用exit(7),WEXITSTATUS(status)就会返回7。
最后写了一段小的测试程序。下面 这段代码在2.6内核的系统上总是返回-1,但是奇怪的是在2.4内核的系统上返回0.
#include
using namespace std;
#include
#include
#include
#include
#include
int main()
{
cout<<"pid: "<<getpid()<<endl;
int ret = 0;
signal(SIGCHLD, SIG_IGN);
ret = system("./wyf.sh");
cout<<"ret: "<<ret<<endl;
return 0;
}
原文链接:http://blog.chinaunix.net/uid-12274566-id-3508850.html
另外,贴上一段自己封装的,能够获取system正确返回值的函数
int my_system(const char *cmd)
{
{
int status = 0;
int ret = NGLOG_SUCCESS;
struct sigaction act, old_act;
act.sa_handler = SIG_DFL;
sigemptyset(&act.sa_mask);
act.sa_flags = 0;
sigaction(SIGCHLD, &act, &old_act);
status = system(cmd);
sigaction(SIGCHLD, &old_act, NULL);
if (-1 == status) {
_LOG_ERR("system: %s", strerror(errno));
ret = -1;
struct sigaction act, old_act;
act.sa_handler = SIG_DFL;
sigemptyset(&act.sa_mask);
act.sa_flags = 0;
sigaction(SIGCHLD, &act, &old_act);
status = system(cmd);
sigaction(SIGCHLD, &old_act, NULL);
if (-1 == status) {
_LOG_ERR("system: %s", strerror(errno));
ret = -1;
} else {
if (WIFEXITED(status)) {
ret =
WEXITSTATUS(status);
if (0 != ret) {
printf("run shell script fail, script exit code: %d", ret);
}
}else{
printf("exit status = %d", WEXITSTATUS(status));
ret = -1;
}
}
return ret;
}
}
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <fcntl.h>
#include <time.h>
#include <signal.h>
#include <termios.h>
#include <unistd.h>
#include <sys/socket.h>
//#include <sys/un.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <sys/wait.h>
#include <sys/resource.h>
#include <linux/kernel.h>
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <fcntl.h>
#include <time.h>
#include <signal.h>
#include <termios.h>
#include <unistd.h>
#include <sys/socket.h>
//#include <sys/un.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <sys/wait.h>
#include <sys/resource.h>
#include <linux/kernel.h>