Bug 781336

Summary: rhn-register GUI won't check for invalid login/password combination
Product: Red Hat Enterprise Linux 5 Reporter: Lukas Pramuk <lpramuk>
Component: rhn-client-toolsAssignee: Stephen Herr <sherr>
Status: CLOSED ERRATA QA Contact: Martin Korbel <mkorbel>
Severity: medium Docs Contact:
Priority: high    
Version: 5.8CC: jhutar, jpazdziora, lkardos, lmiksik, mkorbel, mminar, msuchy
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: rhn-client-tools-0.4.20-78.el5 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 771167 Environment:
Last Closed: 2013-01-08 07:45:07 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 771167    
Bug Blocks: 807971    

Comment 5 RHEL Program Management 2012-04-02 10:45:53 UTC
This request was evaluated by Red Hat Product Management for inclusion
in a Red Hat Enterprise Linux release.  Product Management has
requested further review of this request by Red Hat Engineering, for
potential inclusion in a Red Hat Enterprise Linux release for currently
deployed products.  This request is not yet committed for inclusion in
a release.

Comment 8 Jan Pazdziora (Red Hat) 2012-07-13 14:50:38 UTC
A patch like this could be the least invasive. The reserveUser no longer validates login and password against hosted, while getPossibleOrgs does.

===================================================================
--- src/up2date_client/rhnregGui.py	(revision 208757)
+++ src/up2date_client/rhnregGui.py	(working copy)
@@ -525,6 +525,8 @@
             self.alreadyRegistered = 1
             self.alreadyRegistered = rhnreg.reserveUser(self.loginUname.get_text(),
                                                         self.loginPw.get_text())
+            rhnreg.getPossibleOrgs(self.loginUname.get_text(),
+                                                        self.loginPw.get_text())
         except up2dateErrors.ValidationError, e:
             setArrowCursor()
             self.alreadyRegistered = 0

Comment 10 Stephen Herr 2012-07-13 20:09:28 UTC
*** Bug 783987 has been marked as a duplicate of this bug. ***

Comment 11 Stephen Herr 2012-07-13 20:11:03 UTC
Note that for completeness's sake when we test this we should also test firstboot like in bug 783987

Comment 16 errata-xmlrpc 2013-01-08 07:45:07 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2013-0104.html