Opened 13 years ago

Last modified 13 years ago

#1311 closed Bug report

MFMT Date validation bug

Reported by: sf-steve Owned by:
Priority: normal Component: FileZilla Server
Keywords: Cc: sf-steve, Tim Kosse
Component version: Operating system type:
Operating system version:

Description

MFMT does not allow the hour to be zero. Also it allows it to be more than 23.

The problem is in ControlSocket.cpp where the hour is checked to be within 1 to 31 inclusive versus 0 to 23 inclusive.

Here is the wrong code:

if (year < 1000
month < 1 month > 12 day < 1 day > 31 hour < 1 hour > 31 minute > 59 second > 59)

{
Send( _T("501 Not a valid date") );
break;
}

Change History (1)

comment:1 by Tim Kosse, 13 years ago

Fixed in 0.9.24

Note: See TracTickets for help on using tickets.