Re: [PATCH] Fail the test if number of supported attrs are more than expected.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Sorry for the delay.

Returning unrequested attributes sounds like unlikely behavior, but may
as well check for it, sure....  Applying.

--b.

On Thu, Aug 14, 2014 at 11:02:26AM +0530, Srikrishan Malik wrote:
> Signed-off-by: Srikrishan Malik <srimalik@xxxxxxxxxx>
> ---
>  nfs4.0/servertests/st_getattr.py |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/nfs4.0/servertests/st_getattr.py b/nfs4.0/servertests/st_getattr.py
> index a55b923..6a4acdd 100644
> --- a/nfs4.0/servertests/st_getattr.py
> +++ b/nfs4.0/servertests/st_getattr.py
> @@ -43,10 +43,14 @@ def _try_empty(t, c, path):
>  def _try_supported(t, env, path):
>      c = env.c1
>      mandatory = sum([attr.mask for attr in env.attr_info if attr.mandatory])
> +    all = sum([attr.mask for attr in env.attr_info])
>      supported = c.supportedAttrs(path)
>      if mandatory & supported != mandatory:
>          t.fail("GETATTR(FATTR4_SUPPORTED_ATTRS) did not return "
>                 "all mandatory attributes")
> +    if supported > all:
> +        t.fail("GETATTR(FATTR4_SUPPORTED_ATTRS) returned more than expected "
> +               "supported attributes for the protocol")
>  
>  def _try_long(env, path):
>      c = env.c1
> -- 
> 1.7.1
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux