Markus Elfring <Markus.Elfring@xxxxxx> wrote: > > sysnames should be freed after refcnt being decreased to zero in > > afs_put_sysnames(). > > How do you think about a wording variant like the following? > > Release the sysnames object after its reference counter was decreased > to zero. I would say "reference count" not "reference counter" personally. I would also say "afs_sysnames" rather than "sysnames". 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. > Will it matter to mention the size of the data structure "afs_sysnames"? Why is it necessary to do so? David