自己用strncpy编的程序！

aaron_xueli

程序如下：
  1 #include <stdio.h>
  2 #include <string.h>
  3 #include <stdlib.h>
  4
  5 int main(int argc, char **argv)
  6 {
  7         char *buf = (char *)malloc(strlen(argv[1]));
  8         strncpy(buf, argv[1], strlen(argv[1]));
  9         printf("buf=%s\n", buf);
10
11 }

总感觉这个程序有问题，但不知道问题在哪，请各位支招啦！

cobras

1. 
   
2.      #include <assert.h>
   
3.   1 #include <stdio.h>
   
4.   2 #include <string.h>
   
5.   3 #include <stdlib.h>
   
6.   4
   
7.   5 int main(int argc, char **argv)
   
8.   6 {
   
9.   7         char *buf = (char *)malloc(strlen(argv[1])+1);
   
10.              assert(buf != NULL);
    
11.   8         strcpy(buf, argv[1]);
    
12.   9         printf("buf=%s\n", buf);
    
13.              free(buf);
    
14. 10         return 0;
    
15. 11 }
    

复制代码

ddgfff

malloc(strlen(argv[1])+1)
好像应该是这样吧~

或者用strdup函数:
buf = strdup(arg[1]);

最后记得要
free(buf);

Cha_SARS

strncpy(buf, argv[1], strlen(argv[1]));

copy的长度只是字符串的字符数 后面还有字符串结束'\0'
所以

printf("buf=%s\n", buf);

输出可能会除了argv[1]里的字符外 还有其他 直到碰到0或结束符 当然如果buf申请的内存初始化都为0的话 输出正确的

brightsun__

楼上的strncpy不好吧，没有拷进'\0'，会出问题的

ilwmin

没有初始化,最后没有释放内存,其它没问题.

noword2k

先要判断argc >= 2

aaron_xueli

谢谢各位！！！
:wink:

wangweibd04

这个字符串没有\0结尾可能。保险点还是保证strncpy(desc,src..) 要确保desc的长度大于SRC。而且保险的写法是desc[length-1]='\0';