From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Kurz Subject: Re: [PATCH v2 2/3] Check the return value of fcntl in qemu_set_cloexec Date: Tue, 9 May 2017 22:31:04 +0200 Message-ID: <20170509223104.55412e1b@bahia> References: <1494356693-13190-1-git-send-email-sstabellini@kernel.org> <1494356693-13190-2-git-send-email-sstabellini@kernel.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============7116460136904938100==" Return-path: In-Reply-To: <1494356693-13190-2-git-send-email-sstabellini@kernel.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Stefano Stabellini Cc: pbonzini@redhat.com, Eric Blake , qemu-devel@nongnu.org, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org --===============7116460136904938100== Content-Type: multipart/signed; micalg=pgp-sha1; boundary="Sig_/rwhgYiVV_ui593pL0sJBdi6"; protocol="application/pgp-signature" --Sig_/rwhgYiVV_ui593pL0sJBdi6 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Tue, 9 May 2017 12:04:52 -0700 Stefano Stabellini wrote: > Assert that the return value is not an error. This issue was found by > Coverity. >=20 > CID: 1374831 >=20 > Signed-off-by: Stefano Stabellini > CC: groug@kaod.org > CC: pbonzini@redhat.com > CC: Eric Blake > --- > util/oslib-posix.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) >=20 Reviewed-by: Greg Kurz > diff --git a/util/oslib-posix.c b/util/oslib-posix.c > index 4d9189e..16894ad 100644 > --- a/util/oslib-posix.c > +++ b/util/oslib-posix.c > @@ -182,7 +182,9 @@ void qemu_set_cloexec(int fd) > { > int f; > f =3D fcntl(fd, F_GETFD); > - fcntl(fd, F_SETFD, f | FD_CLOEXEC); > + assert(f !=3D -1); > + f =3D fcntl(fd, F_SETFD, f | FD_CLOEXEC); > + assert(f !=3D -1); > } > =20 > /* --Sig_/rwhgYiVV_ui593pL0sJBdi6 Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iEYEARECAAYFAlkSJwgACgkQAvw66wEB28Jf9wCdEkzkfWxjbcJ1BpUwpEuD1ebM VDYAn02VUPTG59b9dNO1W2vgVw6hbezM =5nY4 -----END PGP SIGNATURE----- --Sig_/rwhgYiVV_ui593pL0sJBdi6-- --===============7116460136904938100== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwczovL2xpc3RzLnhlbi5v cmcveGVuLWRldmVsCg== --===============7116460136904938100==--