Commit 7faafe60 authored by Lukasz Marek's avatar Lukasz Marek

ftp: fix using uninitialized value

Fix coverity issue CID 1026777
Signed-off-by: 's avatarLukasz Marek <lukasz.m.luki@gmail.com>
parent fca435fe
...@@ -265,7 +265,7 @@ static int ftp_auth(FTPContext *s) ...@@ -265,7 +265,7 @@ static int ftp_auth(FTPContext *s)
static int ftp_passive_mode(FTPContext *s) static int ftp_passive_mode(FTPContext *s)
{ {
char *res = NULL, *start, *end; char *res = NULL, *start = NULL, *end = NULL;
int i; int i;
const char *command = "PASV\r\n"; const char *command = "PASV\r\n";
const int pasv_codes[] = {227, 501, 0}; /* 501 is incorrect code */ const int pasv_codes[] = {227, 501, 0}; /* 501 is incorrect code */
...@@ -273,8 +273,7 @@ static int ftp_passive_mode(FTPContext *s) ...@@ -273,8 +273,7 @@ static int ftp_passive_mode(FTPContext *s)
if (ftp_send_command(s, command, pasv_codes, &res) != 227 || !res) if (ftp_send_command(s, command, pasv_codes, &res) != 227 || !res)
goto fail; goto fail;
start = NULL; for (i = 0; res[i]; ++i) {
for (i = 0; i < strlen(res); ++i) {
if (res[i] == '(') { if (res[i] == '(') {
start = res + i + 1; start = res + i + 1;
} else if (res[i] == ')') { } else if (res[i] == ')') {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment