<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Nov 23, 2021 at 3:20 PM Ján Tomko <<a href="mailto:jtomko@redhat.com">jtomko@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Fixes: 59c1bc3a0e25e6d725db41990f11e0b53137115d<br>
Fixes: 43820e4b8037680ec451761216750c6b139db67a<br>
Fixes: 600f580d623ae4077ddeb6c7cb24f8a315a7c73b<br>
Signed-off-by: Ján Tomko <<a href="mailto:jtomko@redhat.com" target="_blank">jtomko@redhat.com</a>><br>
---<br>
 tests/virpcivpdtest.c | 20 ++++++++++++++++++++<br>
 1 file changed, 20 insertions(+)<br>
<br>
diff --git a/tests/virpcivpdtest.c b/tests/virpcivpdtest.c<br>
index 284350fe29..62c51cdeb9 100644<br>
--- a/tests/virpcivpdtest.c<br>
+++ b/tests/virpcivpdtest.c<br>
@@ -446,6 +446,8 @@ testVirPCIVPDReadVPDBytes(const void *opaque G_GNUC_UNUSED)<br>
     buf = g_malloc0(dataLen);<br>
<br>
     fd = virCreateAnonymousFile(fullVPDExample, dataLen);<br>
+    if (fd < 0)<br>
+        return -1;<br></blockquote><div><br></div><div>I would prefer if you rewrote this before merging as:</div><div><br></div><div> <span style="font-family:monospace"><span style="color:rgb(0,0,0);background-color:rgb(255,255,255)">    </span>if<span style="color:rgb(0,0,0);background-color:rgb(255,255,255)"> ((fd = virCreateAnonymousFile(fullVPDExample, dataLen)) < </span><span style="color:rgb(175,95,0);background-color:rgb(255,255,255)">0</span><span style="color:rgb(0,0,0);background-color:rgb(255,255,255)">)
</span><br></span></div><div><span style="font-family:monospace">        return<span style="color:rgb(0,0,0);background-color:rgb(255,255,255)"> -</span><span style="color:rgb(175,95,0);background-color:rgb(255,255,255)">1</span><span style="color:rgb(0,0,0);background-color:rgb(255,255,255)">;</span><br></span></div><div><span style="font-family:monospace"></span><br></div><div>(The whole patch.) I think it looks cleaner, but that's just my preference.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
     readBytes = virPCIVPDReadVPDBytes(fd, buf, dataLen, 0, &csum);<br>
<br>
@@ -482,6 +484,9 @@ testVirPCIVPDParseVPDStringResource(const void *opaque G_GNUC_UNUSED)<br>
<br>
     dataLen = G_N_ELEMENTS(stringResExample);<br>
     fd = virCreateAnonymousFile(stringResExample, dataLen);<br>
+    if (fd < 0)<br>
+        return -1;<br>
+<br>
     result = virPCIVPDParseVPDLargeResourceString(fd, 0, dataLen, &csum, res);<br>
     VIR_FORCE_CLOSE(fd);<br>
<br>
@@ -552,6 +557,9 @@ testVirPCIVPDParseFullVPD(const void *opaque G_GNUC_UNUSED)<br>
<br>
     dataLen = G_N_ELEMENTS(fullVPDExample);<br>
     fd = virCreateAnonymousFile(fullVPDExample, dataLen);<br>
+    if (fd < 0)<br>
+        return -1;<br>
+<br>
     res = virPCIVPDParse(fd);<br>
     VIR_FORCE_CLOSE(fd);<br>
<br>
@@ -620,6 +628,9 @@ testVirPCIVPDParseZeroLengthRW(const void *opaque G_GNUC_UNUSED)<br>
<br>
     dataLen = G_N_ELEMENTS(fullVPDExample);<br>
     fd = virCreateAnonymousFile(fullVPDExample, dataLen);<br>
+    if (fd < 0)<br>
+        return -1;<br>
+<br>
     res = virPCIVPDParse(fd);<br>
     VIR_FORCE_CLOSE(fd);<br>
<br>
@@ -670,6 +681,9 @@ testVirPCIVPDParseNoRW(const void *opaque G_GNUC_UNUSED)<br>
<br>
     dataLen = G_N_ELEMENTS(fullVPDExample);<br>
     fd = virCreateAnonymousFile(fullVPDExample, dataLen);<br>
+    if (fd < 0)<br>
+        return -1;<br>
+<br>
     res = virPCIVPDParse(fd);<br>
     VIR_FORCE_CLOSE(fd);<br>
<br>
@@ -723,6 +737,9 @@ testVirPCIVPDParseFullVPDSkipInvalidKeywords(const void *opaque G_GNUC_UNUSED)<br>
<br>
     dataLen = G_N_ELEMENTS(fullVPDExample);<br>
     fd = virCreateAnonymousFile(fullVPDExample, dataLen);<br>
+    if (fd < 0)<br>
+        return -1;<br>
+<br>
     res = virPCIVPDParse(fd);<br>
     VIR_FORCE_CLOSE(fd);<br>
<br>
@@ -776,6 +793,9 @@ testVirPCIVPDParseFullVPDSkipInvalidValues(const void *opaque G_GNUC_UNUSED)<br>
<br>
     dataLen = G_N_ELEMENTS(fullVPDExample);<br>
     fd = virCreateAnonymousFile(fullVPDExample, dataLen);<br>
+    if (fd < 0)<br>
+        return -1;<br>
+<br>
     res = virPCIVPDParse(fd);<br>
     VIR_FORCE_CLOSE(fd);<br>
<br>
-- <br>
2.31.1<br>
<br></blockquote><div><br></div><div><br></div><div>Kristína <br></div></div></div>