ftp: fix using uninitialized value
authorLukasz Marek <lukasz.m.luki@gmail.com>
Tue, 4 Jun 2013 11:03:13 +0000 (13:03 +0200)
committerLukasz Marek <lukasz.m.luki@gmail.com>
Tue, 4 Jun 2013 11:08:23 +0000 (13:08 +0200)
Fix coverity issue CID 1026777

Signed-off-by: Lukasz Marek <lukasz.m.luki@gmail.com>
libavformat/ftp.c

index cd3006b..faa82dd 100644 (file)
@@ -265,7 +265,7 @@ static int ftp_auth(FTPContext *s)
 
 static int ftp_passive_mode(FTPContext *s)
 {
-    char *res = NULL, *start, *end;
+    char *res = NULL, *start = NULL, *end = NULL;
     int i;
     const char *command = "PASV\r\n";
     const int pasv_codes[] = {227, 501, 0}; /* 501 is incorrect code */
@@ -273,8 +273,7 @@ static int ftp_passive_mode(FTPContext *s)
     if (ftp_send_command(s, command, pasv_codes, &res) != 227 || !res)
         goto fail;
 
-    start = NULL;
-    for (i = 0; i < strlen(res); ++i) {
+    for (i = 0; res[i]; ++i) {
         if (res[i] == '(') {
             start = res + i + 1;
         } else if (res[i] == ')') {