RE: [EXT] Re: [PATCH] test/security: fix buffer leaks in error path

[Akhil Goyal]

> > +#define FREE_PKTS(j, m) {          \
> > +   while (j--)                     \
> > +           rte_pktmbuf_free(m[j]); \
> > +}
> > +
> 
> Changes looks good. However do you want to define a static inline
> function here instead of macro?
Any specific reason for changing to inline function?
In this case macro is as good as inline function.