ftp: explicit return code checks
Signed-off-by: Lukasz Marek <lukasz.m.luki@gmail.com>
This commit is contained in:
		
							parent
							
								
									43eda88200
								
							
						
					
					
						commit
						bc29acdc76
					
				@ -386,7 +386,7 @@ static int ftp_store(FTPContext *s)
 | 
				
			|||||||
    const int stor_codes[] = {150, 0};
 | 
					    const int stor_codes[] = {150, 0};
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    snprintf(command, sizeof(command), "STOR %s\r\n", s->path);
 | 
					    snprintf(command, sizeof(command), "STOR %s\r\n", s->path);
 | 
				
			||||||
    if (!ftp_send_command(s, command, stor_codes, NULL))
 | 
					    if (ftp_send_command(s, command, stor_codes, NULL) != 150)
 | 
				
			||||||
        return AVERROR(EIO);
 | 
					        return AVERROR(EIO);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    s->state = UPLOADING;
 | 
					    s->state = UPLOADING;
 | 
				
			||||||
@ -443,7 +443,7 @@ static int ftp_connect_control_connection(URLContext *h)
 | 
				
			|||||||
        }
 | 
					        }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        /* consume all messages from server */
 | 
					        /* consume all messages from server */
 | 
				
			||||||
        if (!ftp_status(s, NULL, connect_codes)) {
 | 
					        if (ftp_status(s, NULL, connect_codes) != 220) {
 | 
				
			||||||
            av_log(h, AV_LOG_ERROR, "FTP server not ready for new users\n");
 | 
					            av_log(h, AV_LOG_ERROR, "FTP server not ready for new users\n");
 | 
				
			||||||
            err = AVERROR(EACCES);
 | 
					            err = AVERROR(EACCES);
 | 
				
			||||||
            return err;
 | 
					            return err;
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user