• Sergey Poromov's avatar
    Fix using |delayed_login_failure_| after free. · 420e9a82
    Sergey Poromov authored
    AuthFailure was remembered in OnAuthFailure() by a pointer and was
    freed after that leading to strange failure reason on next read.
    It was introduced 5 years ago in https://crrev.com/22900017
    
    [719:719:0918/112724.240220:ERROR:auth_status_consumer.h(95)] NOTREACHED() hit.  [Reason: info:NOTREACHED()]
    [719:719:0918/112724.240293:ERROR:device_event_log_impl.cc(159)] [11:27:24.240] Login: cryptohome_authenticator.cc:725 Login failed:
    [719:719:0918/112724.240371:ERROR:login_performer.cc(63)] Login failure, reason=0, error.state=0
    [719:719:0918/112724.240414:ERROR:kiosk_profile_loader.cc(202)] NOTREACHED() hit.  [Reason: info:NOTREACHED()]
    [719:719:0918/112724.240458:ERROR:auth_status_consumer.h(95)] NOTREACHED() hit.  [Reason: info:NOTREACHED()]
    [719:719:0918/112724.240445:ERROR:kiosk_profile_loader.cc(188)] Kiosk auth failure: error=
    [719:719:0918/112724.240674:ERROR:kiosk_profile_loader.cc(50)] NOTREACHED() hit.  [Reason: info:NOTREACHED()]
    [719:719:0918/112724.240732:ERROR:app_launch_controller.cc(526)] Kiosk launch failed, error=4
    
    Bug: 885346
    Change-Id: I46ca3b3630aa42e25a25d9cfecb3a16e3c361b9f
    Reviewed-on: https://chromium-review.googlesource.com/1236265Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
    Commit-Queue: Sergey Poromov <poromov@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#592830}
    420e9a82
cryptohome_authenticator.cc 43.6 KB