这几天调程序(嵌入式linux),发现程序有时就莫名其妙的死掉,每次都定位在程序中不同的system()函数,直接在shell下输入system()函数中调用的命令也都一切正常,就没理这个bug,以为是其他的代码影响到这个,或是内核驱动文件系统什么的异常导致,昨天又出现了这个问题,就随手百了一下度,问题出现了,很多人都说system()函数要慎用要少用要能不用则不用,system()函数不稳定?
下面对system函数做一个简单的介绍:
头文件
#include
定义函数
int system(const char * string);
函数说明
system()会调用fork()产生子进程,由子进程来调用/bin/sh-c string来执行参数string字符串所代表的命令,此命令执行完后随即返回原调用的进程。在调用system()期间SIGCHLD 信号会被暂时搁置,SIGINT和SIGQUIT 信号则会被忽略。 返回值 =-1:出现错误 =0:调用成功但是没有出现子进程 >0:成功退出的子进程的id 如果system()在调用/bin/sh时失败则返回127,其他失败原因返回-1。若参数string为空指针(NULL),则返回非零值。如果system()调用成功则最后会返回执行shell命令后的返回值,但是此返回值也有可能为 system() 调用/bin/sh失败所返回的127,因此最好能再检查errno 来确认执行成功。
附加说明
在编写具有SUID/SGID权限的程序时请勿使用system(),system()会继承环境变量,通过环境变量可能会造成系统安全的问题。 system函数已经被收录在标准c库中,可以直接调用,使用system()函数调用系统命令的基本使用方法如下:
#include
int main()
{
system("mkdir $HOME/.SmartPlatform/");
system("mkdir $HOME/.SmartPlatform/Files/");
system("cp mainnew.cpp $HOME/.SmartPlatform/Files/");
return 0;
}
下面我们来看看system函数的源码:
#include <>
#include <>
#include <>
int system(const char * cmdstring)
{
pid_t pid;
int status;
if(cmdstring == NULL)
{
return (1);
}
if((pid = fork())<0)
{
status = -1;
}
else if(pid == 0)
{
execl("/bin/sh", "sh", "-c", cmdstring, (char *)0);
_exit(127);
}
else
{
while(waitpid(pid, &status, 0) < 0)
{
if(errno != EINTR)
{
status = -1;
break;
}
}
}
return status;
}
花了两天时间仔细研究了一下,在网上发现了一篇精品博客,介绍的很详细了,谢谢博主,直接转,原文如下:
题:【C/C++】Linux下使用system()函数一定要谨慎
地址:
http://my.oschina.net/renhc/blog/53580
下面是第二篇,对于system()函数的错误详细分析,再次感谢博主。
题:【C/C++】Linux下system()函数引发的错误
地址:
http://my.oschina.net/renhc/blog/54582
继续转该牛X博主的博客,对于上文提到的system()函数的替换函数popen()的详细介绍…万分感谢博主:
题:【IPC通信】基于管道的popen和pclose函数
地址:
http://blog.csdn.net/wangtingyao1990/article/details/50380924
但是根据上面那位博主说的使用system()函数前把SIGCHLD信号处理方式显式修改为SIG_DFL方式,同时记录原来的处理方式,使用完system()后再设为原来的处理方式后,程序还是会死掉。而且看不到system的返回值是多少(因为system在执行系统命令的时候,程序已经挂掉了),故暂时使用博主提到的第二种解决方式使用popen()函数替代system()函数。修改后的函数如下:
int my_system(const char *cmd)
{
FILE * fp;
int res; char buf[1024];
if (cmd == NULL)
{
printf("my_system cmd is NULL!
");
return -1;
}
if ((fp = popen(cmd, "r") ) == NULL)
{
perror("popen");
printf("popen error: %s
", strerror(errno));
return -1;
}
else
{
while(fgets(buf, sizeof(buf), fp))
{
printf("%s", buf);
}
if ( (res = pclose(fp)) == -1 )
{
printf("close popen file pointer fp error!
");
return res;
}
else if (res == 0)
{
return res;
}
else
{
printf("popen res is :%d
", res);
return res;
}
}
}
此时调用my_system()来执行system函数的功能(my_system函数中是使用popen()函数来实现的),测试了一天,没有再次出现程序突然死掉的问题(修改前连续循环调用system()函数测试,每10次就会至少导致程序挂掉一次。连续不停顿的调用)。以上是我对这个问题的总结,先做个记录,待修复bug后再回来仔细研究。
原文地址:
http://blog.sina.com.cn/s/blog_8043547601017qk0.html