Mohamed Ayman created a merge request: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1355

Project:Branches: 
mohamedayman23/rtems:Parameter-forever-is-ignored-with-login-function-Issue to 
rtems/rtos/rtems:main
Author:   Mohamed Ayman



## Fix_Issue_1828: Respect forever flag on failed login

The shell main loop used a single result variable both to record whether the 
previous login/command attempt succeeded and to decide whether the loop should 
continue.

When the login function failed after repeated incorrect passwords, result 
became false, and the loop condition then exited the shell regardless of the 
forever flag being set.

Track login failures separately with a new login_failed flag, and use it 
alongside result in the loop condition so that a failed login correctly falls 
through to a new login attempt when forever is set, matching the documented 
behavior of that flag. Other exit paths EOF, explicit logout and exit are 
unaffected.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1355
You're receiving this email because of your account on gitlab.rtems.org. 
Unsubscribe from this thread: 
https://gitlab.rtems.org/-/sent_notifications/4-4t6tyk78saa7jsv1vgcs09qil-1d/unsubscribe
 | Manage all notifications: https://gitlab.rtems.org/-/profile/notifications | 
Help: https://gitlab.rtems.org/help


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to