下面C程序的预期行为是将自己的可执行文件复制到一个新的随机命名文件,然后执行该文件,这是一个令人讨厌的问题。这应该创建许多可执行文件的副本。这显然是一个可怕的想法,但它仍然是我想要做的。
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/stat.h>
int main(int argc, char* argv[]) {
/* Obtain name of current executable */
const char* binName = argv[0];
/* Print message */
printf("Hello from %s!\n", binName);
/* Create name of new executable */
char newBinName[] = "tmpXXXXXX";
mkstemp(newBinName);
/* Determine size of current executable */
struct stat st;
stat(binName, &st);
const int binSize = st.st_size;
/* Copy current executable to memory */
char* binData = (char*) malloc(sizeof(char) * binSize);
FILE* binFile = fopen(binName, "rb");
fread(binData, sizeof(char), binSize, binFile);
fclose(binFile);
/* Write executable in memory to new file */
binFile = fopen(newBinName, "wb");
fwrite(binData, sizeof(char), binSize, binFile);
fclose(binFile);
/* Make new file executable */
chmod(newBinName, S_IRUSR | S_IWUSR |S_IXUSR);
/* Run new file executable */
execve(
newBinName,
(char*[]) {
newBinName,
NULL
},
NULL);
/* If this code runs, then there has been an error. */
perror("execve");
return EXIT_FAILURE;
}
相反,输出如下:
Hello from ./execTest
execve: Text file busy
我认为文本文件是&#34;忙&#34;因为./execTest
仍在访问它...但我关闭了该文件的文件流。我做错了什么?
答案 0 :(得分:4)
On success, these functions return the file descriptor of the
temporary file.
您放弃mkstemp
返回给您的文件描述符,有效地泄露它。可能它是一个可写的文件描述符,因此会导致execve
失败ETXTBSY
(当有可写的fds打开时会发生)。您是否可以对close()
的返回值尝试mkstemp
并查看是否可以改善行为?
一般反馈点:用C编码时,你应该养成查看返回值的习惯。未能遵守其含义和错误状态通常表示存在错误。