Dan Kenigsberg has posted comments on this change.
Change subject: BZ#725540 vdsClient: check return code first ......................................................................
Patch Set 1: Verified
(1 inline comment)
.................................................... Commit Message Line 3: AuthorDate: 2011-10-13 23:09:35 +0200 Line 4: Commit: Dan Kenigsberg danken@redhat.com Line 5: CommitDate: 2011-10-13 23:21:29 +0200 Line 6: Line 7: BZ#725540 vdsClient: check return code first that bz mentioned one case of unchecked code value. I've fixed other occasions, too. Line 8: Line 9: client should not assume that its data exists in the response before verifying Line 10: that the request was served without error. Line 11:
-- To view, visit http://gerrit.usersys.redhat.com/1031 To unsubscribe, visit http://gerrit.usersys.redhat.com/settings
Gerrit-MessageType: comment Gerrit-Change-Id: Iffa0c0eb6b50b321f97a578994d7cf48bdae2974 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com
vdsm-patches@lists.fedorahosted.org