Change in ffilz/nfs-ganesha[next]: Handle race while adding host to netgroup cache
by Sachin Punadikar (GerritHub)
Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/438804
Change subject: Handle race while adding host to netgroup cache
......................................................................
Handle race while adding host to netgroup cache
Multiple threads may call nfs_ip_name_add function to add the missing
host info to the netgroup cache. Only one thread will be successful
to add that entry and overall access check will be success. For other
threads the access denied error will be set.
Added check so that success is returned for HASHTABLE_OVERWRITTEN or
HASHTABLE_ERROR_KEY_ALREADY_EXISTS errors.
Change-Id: Idc86f6c6f6c458857d418b70200136b7324c8271
Signed-off-by: Sachin Punadikar <psachin(a)in.ibm.com>
---
M src/support/nfs_ip_name.c
1 file changed, 9 insertions(+), 1 deletion(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/04/438804/1
--
To view, visit https://review.gerrithub.io/438804
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: Idc86f6c6f6c458857d418b70200136b7324c8271
Gerrit-Change-Number: 438804
Gerrit-PatchSet: 1
Gerrit-Owner: Sachin Punadikar <psachin(a)in.ibm.com>
5 years, 10 months
Change in ffilz/nfs-ganesha[next]: Don't call nfs_req_creds if we don't have export
by Suhrud Patankar (GerritHub)
Suhrud Patankar has uploaded this change for review. ( https://review.gerrithub.io/438769
Change subject: Don't call nfs_req_creds if we don't have export
......................................................................
Don't call nfs_req_creds if we don't have export
For NLM calls with BAD handle the op_ctx->ctx_export is NULL.
Do not do access checks in this case.
This avoids crash in nfs_req_creds.
Change-Id: Iaa991747cf50fcba3f8ebd57c13b3c33f781f577
Signed-off-by: Suhrud Patankar <suhrudpatankar(a)gmail.com>
---
M src/MainNFSD/nfs_worker_thread.c
1 file changed, 2 insertions(+), 3 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/69/438769/1
--
To view, visit https://review.gerrithub.io/438769
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iaa991747cf50fcba3f8ebd57c13b3c33f781f577
Gerrit-Change-Number: 438769
Gerrit-PatchSet: 1
Gerrit-Owner: Suhrud Patankar <suhrudpatankar(a)gmail.com>
5 years, 10 months