[LTP] [PATCH 1/3] llistxattr/llistxattr01.c: add new testcase

Cyril Hrubis chrubis@suse.cz
Thu Feb 18 13:03:05 CET 2016


Hi!
> > I do not like the special case for seliux here. What we should do
> > instead is to:
> >
> > * Create file/symlink and store it's attribute list
> >
> > * Add an attribute
> >
> > * Check that the list has exactly one more attribute
> >
> > * Remove the file/symlink
> >
> >
> If we don't check that selinux adds default attribute to symlinks, What 
> about the return value.
> 
> We're not going to judge the size of  the extended attribute name list. 
> That's OK?

By "Check that the list has exactly one more attribute" I meant that we
compare the result to the previous state. If we store the attribute list
right after the symlink was created we can check if the increase in list
size is as expected. Or is there a reason why we can't do this?

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the Ltp mailing list