Frank Filz has uploaded this change for review.

View Change

PROXY: Make safer string handling and use display_opaque_bytes_flags

Check return of strlcpy.

Use gsh_strdup where appropriate.

Replace snprintmem with display_opaque_bytes_flags for code re-use.

Change-Id: I49a34fdbeb3498141d4e2f46b4a1c17b74626370
Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>

PROXY: test return of strlcpy

Change-Id: Id85e06d7c5e2b31c63807a8d2fbd714577810c61
Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>

handle_mapping_db: use display functions to make file handle string

Use display_opaque_bytes_flags to produce the same file handle
string as before, re-uses existing code and is safer.

Change-Id: Iedb247a744f1744a721e08ee91735e5d43cf6c3f
Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>
---
M src/FSAL/FSAL_PROXY/handle.c
M src/FSAL/FSAL_PROXY/handle_mapping/handle_mapping_db.c
M src/FSAL/FSAL_PROXY/handle_mapping/test_handle_mapping.c
3 files changed, 64 insertions(+), 65 deletions(-)

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

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

Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: Iedb247a744f1744a721e08ee91735e5d43cf6c3f
Gerrit-Change-Number: 458507
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx@mindspring.com>
Gerrit-MessageType: newchange