On Fri, 1 Mar 2019 16:09:42 +0800 Xiaolong Ye <xiaolong.ye at intel.com> wrote: > + if (umem->buffer) > + free(umem->buffer); Minor nit: you don't need to check for NULL free() already handles this.