Summary: | stuck with sudo and wrong password | ||
---|---|---|---|
Product: | [Frameworks and Libraries] frameworks-kdesu | Reporter: | Harald Sitter <sitter> |
Component: | general | Assignee: | kdelibs bugs <kdelibs-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | de.techno, jr, kdebugs, nate, simonandric5 |
Priority: | NOR | ||
Version: | 5.42.0 | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
See Also: | https://bugs.kde.org/show_bug.cgi?id=377528 | ||
Latest Commit: | https://commits.kde.org/kdesu/2e8ce30c9498b0b3170a82b55ca12277feba21da | Version Fixed In: | 5.55 |
Sentry Crash Report: |
Description
Harald Sitter
2018-01-16 12:05:56 UTC
after looking at the code a bit I am almost entirely certain that the problem is that converseSU doesn't correctly implement sudo handling with su what happens is: - state WaitForPrompt - line="Password: " - write wrong password to stdin - state CheckStar - line="" - line empty - state HandleStub - line=(null) - return notauthorized with sudo - state WaitForPrompt - line="[sudo] password for $USER: " - write wrong password to stdin - state CheckStar - line="" - line empty - state HandleStub - line="Sorry, try again." - stuck: sudo is wayting for a new password, we are waiting for a null line "Sorry, try again." may appear 0-N times depending on sudoers configuration. We cannot influence this specifically. The only way to get anything of the sort going is if we ran sudo with --stdin and closed the fd (causing sudo to only read once and then implode; output would still be divergent from su though). In any case the converseSU parsing needs to be made sudo compatible. rebuilding with missing libudev-dev (We have semi-automated checks for missing build deps but kaffeine doesn't report the missing dep in the normal way so it wasn't picked up) comment was for another bug should be sorted in kaffeine 2.0.15-0xneon+18.04+bionic+build5 wrong bug again Git commit 2e8ce30c9498b0b3170a82b55ca12277feba21da by Jonathan Riddell. Committed on 21/01/2019 at 13:13. Pushed by jriddell into branch 'master'. handle wrong password when using sudo which asks for another password Summary: handle wrong password when using sudo which asks for another password Test Plan: build and run with sudo and su options, test password and no, test diff languages Reviewers: sitter, fvogt Reviewed By: sitter, fvogt Subscribers: starbuck, fvogt, kde-frameworks-devel Tags: #frameworks Differential Revision: https://phabricator.kde.org/D10716 M +4 -0 CMakeLists.txt A +4 -0 autotests/CMakeLists.txt A +2 -0 autotests/config-kdesutest.h.cmake A +98 -0 autotests/kdesutest.cpp [License: LGPL] A +38 -0 autotests/su A +54 -0 autotests/sudo M +10 -3 src/suprocess.cpp https://commits.kde.org/kdesu/2e8ce30c9498b0b3170a82b55ca12277feba21da *** Bug 292176 has been marked as a duplicate of this bug. *** *** Bug 170155 has been marked as a duplicate of this bug. *** |