Solomon Boulos has uploaded this change for review.

View Change

Fix two clang build errors.

clang's static analysis found two "bugs".

The first was an unsigned number (delegation count) was
being checked for being negative (the test should have been == 0).

The second was a gsh_refstr check that the byte array "pointer" was
non NULL. Since the byte array is inline though, the pointer is
always non-NULL (dereferencing the gsh_refstr itself, above, may crash
first).

Change-Id: Ie18344a965074f6b696ed95a652984c939c4a659
Signed-off-by: Solomon Boulos <boulos@google.com>
---
M src/Protocols/NFS/nfs4_pseudo.c
M src/SAL/state_deleg.c
2 files changed, 1 insertion(+), 2 deletions(-)

git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/37/502437/1

To view, visit change 502437. To unsubscribe, or for help writing mail filters, visit settings.