diff options
| author | Lukasz Marek <lukasz.m.luki@gmail.com> | 2013-06-04 13:03:13 +0200 |
|---|---|---|
| committer | Lukasz Marek <lukasz.m.luki@gmail.com> | 2013-06-04 13:08:23 +0200 |
| commit | 7faafe606fb25e3c8b3091ea0565b01622c87dd2 (patch) | |
| tree | 62b40f10ad455c0f78704f6acd5403d6767a9389 | |
| parent | fca435fee167da981f024e35d9fef4e6179b8061 (diff) | |
ftp: fix using uninitialized value
Fix coverity issue CID 1026777
Signed-off-by: Lukasz Marek <lukasz.m.luki@gmail.com>
| -rw-r--r-- | libavformat/ftp.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/libavformat/ftp.c b/libavformat/ftp.c index cd3006b495..faa82dd30d 100644 --- a/libavformat/ftp.c +++ b/libavformat/ftp.c @@ -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] == ')') { |