Dan Kenigsberg has posted comments on this change.

Change subject: cpopen: Fix signal handling in sync points
......................................................................


Patch Set 5: (1 inline comment)

....................................................
File lib/cpopen/cpopen.c
Line 63: 
Line 64:     return rv;
Line 65: }
Line 66: 
Line 67: /* Just like close() but retries on interrupt and tries to fill the 
buffer */
close -> read
Line 68: static int
Line 69: safeRead(int fd, void *buff, size_t count) {
Line 70:     size_t bread = 0;
Line 71:     char* cbuff = buff;


--
To view, visit http://gerrit.ovirt.org/15165
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id2883091c377c1e4ff9f69b87fc137326b583ab8
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to