最近在版子上要從server端下載更新檔來更新
int download_file(char *img[][2],int count){
char cmd[128] = {0};
int i;
//strcat(cmd,"wget -O /data/update.xml'");
for( i=0;i<count;i++){
strcat(cmd,"wget ");
strcat(cmd,"-O /Data/");
strcat(cmd,img[i][0]);
strcat(cmd," ");
strcat(cmd,"\"");
strcat(cmd,url);
strcat(cmd,"?");
strcat(cmd,"&img_name=");
strcat(cmd,img[i][0]);
strcat(cmd,"&token=");
strcat(cmd,token_decrypt);
strcat(cmd,"\"");
system(cmd)
memset(cmd,0,sizeof(cmd));
}
但是發現到下載到的檔案跟原本的檔案大小差很多 而且還會出現segmentation fault
不知道這樣的寫法對不對 想說會不會是同步下載的關係 難道不能等一個下載玩
在接下一個嗎?

你怎麼不用script來做?

要不要把 cmd printf 出來看一下

有沒有可能 128 不夠大？

char *img[][2]有點怪,img ia a array pointers to char[2]所以你每個image name最多就兩個char, 扣掉\0, 剩一個char請問你的檔名是"a"這類的嗎?再來是，請不要用strcat, 那是下流的寫法int len=0; //reset buff to headlen+=snprintf(buff+len,sizeof(buff)-len,format,...);建議把snprintf學起來，strcat真的不要再用了。嗯, 總結: 請愛用script.

原po的char *img[][2]部份應該無錯,img[][]是char *

@dirkc: 你說的對﹐是我誤會成"char img[][2]", 但我還是覺得有可能原po誤用了,一般人不會用到這麼復雜的(char*)[][]

感謝樓上大大的回復 我在試看看 我也覺得strcat 很蠢 所所以也有想到用snprintf