CID#24090 Resource leak in create_log_facility
Signed-off-by: Thomas Favre-Bulle <thomas.favre-bulle@bull.net>
CID#24077 Uninitialized scalar variable in nfs3_write

Eventually, if entry is NULL then execution jumps to the out label where
written_size is passed to server_stats_io_done with an unknown value.
Signed-off-by: Thomas Favre-Bulle <thomas.favre-bulle@bull.net>
Do not grant delegation if lock_op has failed.

Signed-off-by: Srikrishan Malik <srimalik@in.ibm.com>
Unlock before return.

Signed-off-by: Jeremy Bongio <jbongio@us.ibm.com>
Signed-off-by: Srikrishan Malik <srimalik@in.ibm.com>
Set fsal in upcall args.

Signed-off-by: Srikrishan Malik <srimalik@in.ibm.com>
Add fast RPCs counters globally for the server.

Signed-off-by: Marc Eshel <eshel@us.ibm.com>
Defined new option - 'Disable_ACL' in the 'export_perms' list

Signed-off-by: Soumya Koduri <skoduri@redhat.com>
Fix Coverity CID 18439.

Signed-off-by: Marc Eshel <eshel@us.ibm.com>
Coverity 18501.

Signed-off-by: Marc Eshel <eshel@us.ibm.com>
Add DBusError arg to all method handlers

Propagate proper DBus error handling introduced with properties handlers
to methods.  This makes for proper DBus method error handling.

Signed-off-by: Jim Lieb <jlieb@panasas.com>
Fix Cmake warning reported by Dev_16 Coverity analysis

Signed-off-by: Jim Lieb <jlieb@panasas.com>
git hooks: use project checkpatch config file

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
git hooks: make the hook install script work with spaces in file names!

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Fix using ganesha abstract_atomic.h header

We were including RPC headers which may include tirpc'c
abstract_atomic.h file before including our own abstract_atomic.h.
Both header files (ganesha abstract_atomic.h as well as tirpc's
abstract_atomic.h) use same #ifndef macro to include the header.
So our abstract_atomic.h may not be used at all in some files.

Modified src/include/ganesha_rpc.h to include abstract_atomic.h
before including any RPC headers.

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Set fsal_stable bool correctly in GPFSFSAL_write.

bool is a character, so we should not cast fsal_stable to uint32_t
pointer. *fsal_stable was also unchanged, so set it correctly now.

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Fix NLM with new delegation support.

Signed-off-by: Marc Eshel <eshel@us.ibm.com>
Fix checkpatch errors

Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>
Remove unecessary #include "fsal_handle_syscalls.h"

Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>
Add missing COMPONENT_EXPORT to set of configurable options

Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>
Enhance debug of multiple FSALs

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