Fix inverse logic. If this is zone_pack, then we shouldn't free the

cluster ourselves.

Found by review. Since this code is !386 and !amd64 and is executed
on error path, pretty sure no one ever executed it.
This commit is contained in:
glebius 2016-02-03 20:39:52 +00:00
parent 3e3f3a4c0f
commit ba8e82fa42

View File

@ -738,7 +738,7 @@ refill_fl(adapter_t *sc, struct sge_fl *q, int n)
cl, q->buf_size, refill_fl_cb, &cb_arg, 0); cl, q->buf_size, refill_fl_cb, &cb_arg, 0);
if (err != 0 || cb_arg.error) { if (err != 0 || cb_arg.error) {
if (q->zone == zone_pack) if (q->zone != zone_pack)
uma_zfree(q->zone, cl); uma_zfree(q->zone, cl);
m_free(m); m_free(m);
goto done; goto done;