On 15 December 2014 at 06:22, Maxim Uvarov <[email protected]> wrote: > > CID 84870 > Coverity found uninitialized variable in case if > classifier drops packets. > > Signed-off-by: Maxim Uvarov <[email protected]> >
Reviewed-and-tested-by: Mike Holmes <[email protected]> > --- > platform/linux-generic/odp_packet_io.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/platform/linux-generic/odp_packet_io.c > b/platform/linux-generic/odp_packet_io.c > index a03eeb1..067282e 100644 > --- a/platform/linux-generic/odp_packet_io.c > +++ b/platform/linux-generic/odp_packet_io.c > @@ -483,9 +483,12 @@ odp_buffer_hdr_t *pktin_dequeue(queue_entry_t *qentry) > tmp_hdr_tbl[j++] = buf_hdr; > } > > - if (j) > - queue_enq_multi(qentry, tmp_hdr_tbl, j); > + if (0 == j) > + return NULL; > + > + queue_enq_multi(qentry, tmp_hdr_tbl, j); > buf_hdr = tmp_hdr_tbl[0]; > + > return buf_hdr; > } > > -- > 1.8.5.1.163.gd7aced9 > > > _______________________________________________ > lng-odp mailing list > [email protected] > http://lists.linaro.org/mailman/listinfo/lng-odp > -- *Mike Holmes* Linaro Sr Technical Manager LNG - ODP
_______________________________________________ lng-odp mailing list [email protected] http://lists.linaro.org/mailman/listinfo/lng-odp
