Change else case to bcUnknown on Line 733 of BCrypt.pas #2

Closed
opened 2016-04-26 10:54:33 -07:00 by sikofitt · 0 comments
sikofitt commented 2016-04-26 10:54:33 -07:00 (Migrated from github.com)

The else case on #L733 of BCrypt.pas returns bcPHP on the chance 2a or 2y doesn't match. This is wrong. If someone used 2b, 2c, etc. It would return bcPHP and it would be wrong. It should return bcUnknown.

The else case on [#L733](https://github.com/renegadebbs/pascal_bcrypt/blob/master/BCrypt.pas#L733) of BCrypt.pas returns bcPHP on the chance 2a or 2y doesn't match. **_This is wrong._** If someone used 2b, 2c, etc. It would return bcPHP and it would be wrong. It should return [bcUnknown](https://github.com/renegadebbs/pascal_bcrypt/blob/master/BCrypt.pas#L203).
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: eric/pascal_bcrypt#2
No description provided.