我遇到_popen(const char *, const char *)
我正在尝试使用awk从adb获取信息,但我的程序在第二次_popen调用时崩溃(该方法被调用两次)。
这是有问题的代码(sysinf_getvalue()
在主要部分被多次调用。)
int deviceadbinfowritten = 0;
int devicebootloaderinfowritten = 0;
int adbinit = 0;
int initadb()
{
if(!adbinit)
{
system("adb kill-server && adb start-server");
adbinit = 1;
}
return 0;
}
int sysinfo_getadbinfofile()
{
if(!deviceadbinfowritten)
{
system("echo Please connect your device && adb wait-for-device && adb.exe shell getprop > deviceinfo");
deviceadbinfowritten = 1;
}
return 0;
}
int sysinfo_getbootloaderinfofile()
{
if(!devicebootloaderinfowritten)
{
system("adb wait-for-device && adb reboot bootloader && fastboot getvar all > bootloaderinfo && fastboot reboot");
devicebootloaderinfowritten = 1;
}
return 0;
}
char *sysinfo_getvalue(char *key, int bootloader)
{
initadb();
if(bootloader) sysinfo_getbootloaderinfofile();
else sysinfo_getadbinfofile();
char *file = (bootloader)?"bootloaderinfo":"deviceinfo";
char *command = malloc(sizeof(char) * (strlen("awk.exe -F\":\" \" { print $2 }\" < ") + strlen(key) + (bootloader)?0:2 + strlen(file)));
char *adbkey = malloc((strlen(key)+2)* (sizeof(char)));
if(!bootloader) sprintf(adbkey, "/%s/", key);
sprintf(command, "awk.exe -F\":\" \"%s { print $2 }\" < %s", ((bootloader)?key:adbkey), file);
printf("%s\n", command);
char *pointer = exe(command);
if(!bootloader)
{
pointer += 2;
pointer[strlen(pointer)-3] = 0;
}
return pointer;
}
char *exe(char *exe)
{
char buffer[1024];
FILE *f;
printf("debug %s\n", exe);
f = _popen(exe, "r");
printf("debug\n");
char *res = NULL;
printf("debug\n");
if (f == NULL) {
printf("[-] Failed to run command\n" );
return NULL;
}
printf("debug\n");
while (fgets(buffer, sizeof(buffer), f) != NULL) {
if(res == NULL) {res = malloc(strlen(buffer) * sizeof(char)); strcpy(res, buffer);}
else {res = realloc(res, strlen(res) * sizeof(char) + sizeof(buffer));sprintf(res, "%s%s", res, buffer);}
}
printf("debug\n");
_pclose(f);
return res;
}
注意:如果我使用相同的字符串直接调用main中的exe()
_popen不会崩溃。
注2:调试器说"warning: HEAP[program.exe]:
warning: Heap block at 002D2EC0 modified at 002D2EC9 past requested size of 1"
答案 0 :(得分:1)
看起来你错过了为0终结者分配空间。
这是每个字符数组需要的最后一个额外字节,如果用作字符串,用字符值NUL
(十进制0
)来表示字符串的结尾。
至少这一行:
char *command = malloc(sizeof(char) * (strlen("awk.exe -F\":\" \" { print $2 }\" < ") + strlen(key) + (bootloader)?0:2 + strlen(file)));
应再分配一个字符:
char *command = malloc(
sizeof(char) * (
strlen("awk.exe -F\":\" \" { print $2 }\" < ") +
strlen(key) +
(bootloader)?0:2 + strlen(file)) +
1
)
);
(对于这样的问题,我没有检查代码中字符数组的所有分配/停止。)
答案 1 :(得分:0)
我在此链接C++ system() not working when there are spaces in two different parameters中发现,如果您希望它们正常工作,则需要使用引号封装所有系统调用。
例如,在我的代码中,我需要使用exe("\"mycommand\"");
而不是exe("mycommand");