> Perhaps something like:
>
>       Fix afs_put_sysnames() to actually free the specified afs_sysnames
>       object after its reference count has been decreased to zero and its
>       contents have been released.

* How do you think about to omit the word "Fix" because of the provided tag?

* Is freeing and releasing an item a duplicate operation anyhow?


>> Will it matter to mention the size of the data structure "afs_sysnames"?
>
> Why is it necessary to do so?

I suggest to express the impact of the missed function call "kfree".

Regards,
Markus

Reply via email to