----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105895/ -----------------------------------------------------------
(Updated Aug. 8, 2012, 7:59 a.m.) Review request for KDE Base Apps. Changes ------- Updated diff as per email conversation. Description ------- When changing the user's full name, chfn may not necessarily produce any output. Since readLine blocks, the kcm may hang. This change checks if chfn exited without output, and if so, use that exit status. This addresses bug 156396. http://bugs.kde.org/show_bug.cgi?id=156396 Diffs (updated) ----- kdepasswd/kcm/chfnprocess.cpp 9f75d4aa75b41acec84e7798c789d4226ca3fab9 Diff: http://git.reviewboard.kde.org/r/105895/diff/ Testing ------- On a PAM-enabled system: * Full name changed successfully when permitted by login.defs * Error presented and no change processed with prohibited by login.defs Thanks, Michael Palimaka