Page MenuHomeFreeBSD
Files F167519842

D9234.id24172.diff
No OneTemporary
Actions

D9234.id24172.diff
View Options

Index: sys/dev/xen/netback/netback.c
===================================================================
--- sys/dev/xen/netback/netback.c
+++ sys/dev/xen/netback/netback.c
@@ -1101,14 +1101,13 @@
xs_vprintf(XST_NIL, xenbus_get_node(xnb->dev),
"hotplug-error", fmt, ap_hotplug);
va_end(ap_hotplug);
- xs_printf(XST_NIL, xenbus_get_node(xnb->dev),
+ (void)xs_printf(XST_NIL, xenbus_get_node(xnb->dev),
"hotplug-status", "error");
xenbus_dev_vfatal(xnb->dev, err, fmt, ap);
va_end(ap);
- xs_printf(XST_NIL, xenbus_get_node(xnb->dev),
- "online", "0");
+ (void)xs_printf(XST_NIL, xenbus_get_node(xnb->dev), "online", "0");
xnb_detach(xnb->dev);
}
Index: sys/dev/xen/netback/netback_unit_tests.c
===================================================================
--- sys/dev/xen/netback/netback_unit_tests.c
+++ sys/dev/xen/netback/netback_unit_tests.c
@@ -1227,6 +1227,10 @@
xnb_ring2pkt(&pkt, &xnb_unit_pvt.txb, xnb_unit_pvt.txb.req_cons);
pMbuf = xnb_pkt2mbufc(&pkt, xnb_unit_pvt.ifp);
+ XNB_ASSERT(pMbuf != NULL);
+ if (pMbuf == NULL)
+ return;
+
n_entries = xnb_txpkt2gnttab(&pkt, pMbuf, xnb_unit_pvt.gnttab,
&xnb_unit_pvt.txb, DOMID_FIRST_RESERVED);
@@ -1271,8 +1275,7 @@
/* should never get here */
XNB_ASSERT(0);
}
- if (pMbuf != NULL)
- m_freem(pMbuf);
+ m_freem(pMbuf);
}
@@ -1494,15 +1497,14 @@
struct mbuf *mbufc, *mbufc2;
mbufc = m_getm(NULL, size1, M_WAITOK, MT_DATA);
- mbufc->m_flags |= M_PKTHDR;
- if (mbufc == NULL) {
- XNB_ASSERT(mbufc != NULL);
+ XNB_ASSERT(mbufc != NULL);
+ if (mbufc == NULL)
return;
- }
+ mbufc->m_flags |= M_PKTHDR;
mbufc2 = m_getm(mbufc, size2, M_WAITOK, MT_DATA);
+ XNB_ASSERT(mbufc2 != NULL);
if (mbufc2 == NULL) {
- XNB_ASSERT(mbufc2 != NULL);
safe_m_freem(&mbufc);
return;
}
@@ -1537,11 +1539,10 @@
struct mbuf *mbufc, *m;
mbufc = m_getm(NULL, size, M_WAITOK, MT_DATA);
- mbufc->m_flags |= M_PKTHDR;
- if (mbufc == NULL) {
- XNB_ASSERT(mbufc != NULL);
+ XNB_ASSERT(mbufc != NULL);
+ if (mbufc == NULL)
return;
- }
+ mbufc->m_flags |= M_PKTHDR;
mbufc->m_pkthdr.len = size;
size_remaining = size;
@@ -1576,10 +1577,9 @@
struct mbuf *mbufc, *m;
mbufc = m_getm(NULL, size, M_WAITOK, MT_DATA);
- if (mbufc == NULL) {
- XNB_ASSERT(mbufc != NULL);
+ XNB_ASSERT(mbufc != NULL);
+ if (mbufc == NULL)
return;
- }
mbufc->m_flags |= M_PKTHDR;
mbufc->m_pkthdr.len = size;
@@ -1619,11 +1619,10 @@
int error;
mbufc = m_getm(NULL, size, M_WAITOK, MT_DATA);
- mbufc->m_flags |= M_PKTHDR;
- if (mbufc == NULL) {
- XNB_ASSERT(mbufc != NULL);
+ XNB_ASSERT(mbufc != NULL);
+ if (mbufc == NULL)
return;
- }
+ mbufc->m_flags |= M_PKTHDR;
mbufc->m_pkthdr.len = size;
size_remaining = size;
@@ -1840,10 +1839,9 @@
struct netif_extra_info *ext;
mbufc = m_getm(NULL, size, M_WAITOK, MT_DATA);
- if (mbufc == NULL) {
- XNB_ASSERT(mbufc != NULL);
+ XNB_ASSERT(mbufc != NULL);
+ if (mbufc == NULL)
return;
- }
mbufc->m_flags |= M_PKTHDR;
mbufc->m_pkthdr.len = size;
@@ -1974,11 +1972,10 @@
struct mbuf *mbufc;
mbufc = m_getm(NULL, size1, M_WAITOK, MT_DATA);
- mbufc->m_flags |= M_PKTHDR;
- if (mbufc == NULL) {
- XNB_ASSERT(mbufc != NULL);
+ XNB_ASSERT(mbufc != NULL);
+ if (mbufc == NULL)
return;
- }
+ mbufc->m_flags |= M_PKTHDR;
m_getm(mbufc, size2, M_WAITOK, MT_DATA);
XNB_ASSERT(mbufc->m_next != NULL);
@@ -2451,7 +2448,7 @@
for (i = 0; i < 12; i++)
dest[i] = 'X';
- sscanf(mystr, "%hhu", &dest[4]);
+ XNB_ASSERT(sscanf(mystr, "%hhu", &dest[4]) == 1);
for (i = 0; i < 12; i++)
XNB_ASSERT(dest[i] == (i == 4 ? 137 : 'X'));
}
@@ -2469,7 +2466,7 @@
for (i = 0; i < 12; i++)
dest[i] = 'X';
- sscanf(mystr, "%hhd", &dest[4]);
+ XNB_ASSERT(sscanf(mystr, "%hhd", &dest[4]) == 1);
for (i = 0; i < 12; i++)
XNB_ASSERT(dest[i] == (i == 4 ? -27 : 'X'));
}
@@ -2487,7 +2484,7 @@
for (i = 0; i < 3; i++)
dest[i] = (long long)0xdeadbeefdeadbeef;
- sscanf(mystr, "%lld", &dest[1]);
+ XNB_ASSERT(sscanf(mystr, "%lld", &dest[1]) == 1);
for (i = 0; i < 3; i++)
XNB_ASSERT(dest[i] == (i != 1 ? (long long)0xdeadbeefdeadbeef :
-123456789012345));
@@ -2506,7 +2503,7 @@
for (i = 0; i < 3; i++)
dest[i] = (long long)0xdeadbeefdeadbeef;
- sscanf(mystr, "%llu", &dest[1]);
+ XNB_ASSERT(sscanf(mystr, "%llu", &dest[1]) == 1);
for (i = 0; i < 3; i++)
XNB_ASSERT(dest[i] == (i != 1 ? (long long)0xdeadbeefdeadbeef :
12802747070103273189ull));
@@ -2528,10 +2525,10 @@
for (i = 0; i < 12; i++)
dest[i] = (unsigned char)'X';
- sscanf(mystr,
+ XNB_ASSERT(sscanf(mystr,
"000102030405060708090a0b0c0d0e0f101112131415161718191a1b1c1d1e1f"
"202122232425262728292a2b2c2d2e2f303132333435363738393a3b3c3d3e3f"
- "404142434445464748494a4b4c4d4e4f%hhn", &dest[4]);
+ "404142434445464748494a4b4c4d4e4f%hhn", &dest[4]) == 1);
for (i = 0; i < 12; i++)
XNB_ASSERT(dest[i] == (i == 4 ? 160 : 'X'));
}

File Metadata

Mime Type
text/plain
Expires
Sun, Aug 23, 10:55 AM (17 h, 56 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
37091006
Default Alt Text
D9234.id24172.diff (4 KB)

Event Timeline