* Jiri Kosina <jkosina@xxxxxxx> wrote: > + name = kmalloc(sizeof(char) * UDF_NAME_LEN, GFP_KERNEL); > + fname = kmalloc(sizeof(char) * UDF_NAME_LEN, GFP_KERNEL); > + > + if (!name || !fname) { > + *err = -ENOMEM; > + return NULL; > + } > + > if (dentry) { > if (!dentry->d_name.len) { > *err = -EINVAL; this bit is missing i think: if (name) kfree(name); if (fname) kfree(fname); Ingo -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel