Ross Lagerwall [Sun, 20 Jul 2014 07:18:06 +0000 (08:18 +0100)]
libnfs.h: Update open() flags docs
Signed-off-by: Ross Lagerwall <rosslagerwall@gmail.com>
Ross Lagerwall [Sun, 20 Jul 2014 07:13:45 +0000 (08:13 +0100)]
libnfs.c: Handle OOM condition
Signed-off-by: Ross Lagerwall <rosslagerwall@gmail.com>
Ross Lagerwall [Sun, 20 Jul 2014 07:07:46 +0000 (08:07 +0100)]
libnfs.c: Support use of O_TRUNC with nfs_create
Simplify usage for application programmers by supporting the use of
O_TRUNC with nfs_create.
Signed-off-by: Ross Lagerwall <rosslagerwall@gmail.com>
Ronnie Sahlberg [Sat, 19 Jul 2014 23:01:13 +0000 (16:01 -0700)]
Merge pull request #88 from rosslagerwall/76-no-autoreconnect
libnfs.c: Fix autoreconnect
Ronnie Sahlberg [Sat, 19 Jul 2014 22:58:30 +0000 (15:58 -0700)]
Merge pull request #86 from rosslagerwall/75-create
libnfs.c: add nfs_create
Ronnie Sahlberg [Sat, 19 Jul 2014 22:56:59 +0000 (15:56 -0700)]
Merge pull request #87 from rbalint/master
Use sockaddr_in6 on FreeBSD
Ronnie Sahlberg [Sat, 19 Jul 2014 22:47:35 +0000 (15:47 -0700)]
New version 1.9.5
This version removes old ONC-RPC symbols and automatically includes the
RPC/ZDR layer include from the raw low level headers.
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Petr Salinger [Fri, 18 Jul 2014 23:14:43 +0000 (01:14 +0200)]
Use sockaddr_in6 instead of sockaddr6_in
Ross Lagerwall [Sat, 19 Jul 2014 10:32:40 +0000 (11:32 +0100)]
libnfs.c: Fix autoreconnect
Since rpc_connect_async sets autoreconnect to 0, turn autoreconnect on
after the connection has completed, not before.
This fixes #76.
Signed-off-by: Ross Lagerwall <rosslagerwall@gmail.com>
Ross Lagerwall [Thu, 17 Jul 2014 21:31:23 +0000 (22:31 +0100)]
libnfs.c: add nfs_create
Add a new family of functions, nfs_create, like nfs_creat but takes an
additional flags argument which allows extra flags like O_SYNC, O_EXCL
and O_APPEND to be specified.
Ronnie Sahlberg [Sun, 13 Jul 2014 22:36:59 +0000 (15:36 -0700)]
RPC: include libnfs-zdr.h from all raw libnfs-raw-*.h files
Include the zdr header from all raw lowlevel headers so that we don't have to
include it manually from apps.
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 13 Jul 2014 22:20:09 +0000 (15:20 -0700)]
Merge pull request #85 from rsahlberg/remove-zhr.h-dependency
Remove zhr.h dependency
Ronnie Sahlberg [Sat, 5 Jul 2014 19:44:18 +0000 (12:44 -0700)]
ZDR: remove dependency on zdr.h from the examples and nfs-ls
Signed-off-by: Ronnie Sahlberg <sahlberg@localhost>
Ronnie Sahlberg [Sat, 5 Jul 2014 16:07:23 +0000 (09:07 -0700)]
RPC: get rid of bool_t
Signed-off-by: Ronnie Sahlberg <sahlberg@localhost>
Ronnie Sahlberg [Sat, 5 Jul 2014 15:52:46 +0000 (08:52 -0700)]
RPC: get rid of references to SCVXPRT
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sat, 5 Jul 2014 15:45:45 +0000 (08:45 -0700)]
RPC: get rid of all references to CLIENT
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 29 Jun 2014 21:57:45 +0000 (14:57 -0700)]
New version 1.9.4
New version of libnfs:
- IPv6 support
- Support for Portmapper version 3 and an example portmap client.
- Directory caching top make repeated opendir() calls faster
- Readahead support
- Build manpages for the utilities (==nfs-ls)
- Support for O_APPEND
- Rename the list macros to avoid collission on *BSD
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 29 Jun 2014 21:42:28 +0000 (14:42 -0700)]
libnfs.c: Change the directory cache to a smaller more reasonable value
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 29 Jun 2014 21:41:28 +0000 (14:41 -0700)]
Merge pull request #82 from plieven/readahead
add readahead support
Peter Lieven [Mon, 23 Jun 2014 14:31:32 +0000 (16:31 +0200)]
add readahead support
This patch add support for an internal readahead machanism. The maximum readahead
size can be specified via URL parameter readahead. This should significantly
speed up small sequential reads.
Signed-off-by: Peter Lieven <pl@kamp.de>
Ronnie Sahlberg [Sat, 14 Jun 2014 00:03:35 +0000 (17:03 -0700)]
update copyright bilerplates also in generated files
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 8 Jun 2014 18:40:35 +0000 (11:40 -0700)]
libnfs.c: add nlink to nfsdirent so we can get it for 'free'
update the nfs-ls utility to just use nfsdirent as is instead of having to
stat the files.
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 8 Jun 2014 18:22:06 +0000 (11:22 -0700)]
libnfs.c: add a simple dircache to libnfs
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 8 Jun 2014 16:45:22 +0000 (09:45 -0700)]
libnfs.c: always pass the attributes to the callback for recursive lookups
Always pass a fattr3 structure to the callbacks for the internal function
to perform recursive lookups : nfs_lookuppath_async().
This will allow us to access for example the mtime for an object before we
start performing any expensive functions.
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 8 Jun 2014 14:54:45 +0000 (07:54 -0700)]
win32_errnowrapper.h: add licence boilerplate
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 8 Jun 2014 14:53:03 +0000 (07:53 -0700)]
aros: add licence to aros_compat.h
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 8 Jun 2014 14:51:38 +0000 (07:51 -0700)]
proto files: add a simplified bsd licence to the dot-x files
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 8 Jun 2014 14:34:54 +0000 (07:34 -0700)]
init.c: fixup the mangled lgplv2.1 boilerplate
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 8 Jun 2014 14:21:43 +0000 (07:21 -0700)]
init.c: use list macro when removing pdus from the wait list
Use the macro when removing the pdus in the wait list from the queues.
Also make sure to remove them from the right queue, from waitqueue and not
the outqueue for PDUs we have already sent out.
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sun, 8 Jun 2014 13:33:09 +0000 (06:33 -0700)]
Merge pull request #79 from plieven/readahead
socket: fix broken connect for non broadcast traffic
Peter Lieven [Sun, 8 Jun 2014 12:20:12 +0000 (14:20 +0200)]
socket: fix broken connect for non broadcast traffic
commit
1c1e09a completely broke connects for non broadcast
traffic since it forgot to copy the server address into
the socket_storage struct.
Signed-off-by: Peter Lieven <pl@kamp.de>
Ronnie Sahlberg [Thu, 15 May 2014 02:11:03 +0000 (19:11 -0700)]
slist.h: Change SLIST_* to LIBNFS_LIST_* to avoid name clash on *BSD
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Thu, 15 May 2014 02:04:43 +0000 (19:04 -0700)]
docs: add a prebuilt manpage to the distribution
Add a prebuilt manpage for nfs-ls and change the makefiles to not build the
manpage by default. This is because the manpages change rarely and this removes
the need for an internet connection when building the library.
(previously building the library required downloading a template from
sourceforge)
Update the README and mention that you now need to manually rebuild the
manpages if the manpage sources change.
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Ronnie Sahlberg [Sat, 19 Apr 2014 18:38:11 +0000 (11:38 -0700)]
Merge pull request #74 from alexsn/master
nfs-ls: Fix crash when called with no arguments
Alex Snast [Sat, 19 Apr 2014 16:30:43 +0000 (19:30 +0300)]
nfs-ls: Fix crash when called with no arguments
When called with no argument we jump to 'finished' label
after which nfs_destroy_url is called with *url having
some garbage stack value.
Ronnie Sahlberg [Fri, 18 Apr 2014 16:32:01 +0000 (09:32 -0700)]
nfs_open: add support for O_APPEND
Ronnie Sahlberg [Fri, 11 Apr 2014 02:59:28 +0000 (19:59 -0700)]
Merge branch 'xid-hash'
Ronnie Sahlberg [Fri, 11 Apr 2014 01:21:13 +0000 (18:21 -0700)]
Merge pull request #71 from Memphiz/nolinger
[socket] - disable linger by setting SO_LINGER to 0 seconds
Ronnie Sahlberg [Fri, 11 Apr 2014 01:20:52 +0000 (18:20 -0700)]
Merge pull request #72 from Memphiz/fixosxcompile
[osx] - fix compile
Memphiz [Thu, 10 Apr 2014 18:31:25 +0000 (20:31 +0200)]
[osx] - fix compile
Memphiz [Thu, 10 Apr 2014 18:30:30 +0000 (20:30 +0200)]
[socket] - disable linger by setting SO_LINGER to 0 seconds
Arne Redlich [Mon, 31 Mar 2014 21:40:26 +0000 (23:40 +0200)]
Fix the file position handling of the read and write calls
Since the interface is modelled after the libc calls we should try to match
their behaviour to avoid unpleasant surprises:
* read / write (sync and async flavours) update the file position
* pread / pwrite (sync and async flavours) do not update the file position
.
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Arne Redlich [Mon, 31 Mar 2014 21:35:38 +0000 (23:35 +0200)]
libnfs-sync: nfs_{read,write}: use their _async counterparts internally
Previously nfs_read and nfs_write used to use nfs_pread and nfs_pwrite respectively.
In preparation of getting the file position handling right this has to be detangled.
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Arne Redlich [Mon, 31 Mar 2014 19:23:48 +0000 (21:23 +0200)]
nfs_lseek{,_async}: allow negative offsets and guard against file positions < 0
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Mark Hills [Thu, 20 Feb 2014 14:02:08 +0000 (14:02 +0000)]
Track waiting requests in a hash table, by xid
NFS servers can respond to requests in any order, and they do. In our
tests there is also some clustering to the responses; it could be
because eg. requests are served synchronously if the data is in the cache.
Introduce a hash table so that we are able to find the pdu quickly in
all cases, assuming random distribution of the responses.
Mark Hills [Thu, 27 Feb 2014 12:52:54 +0000 (12:52 +0000)]
Ensure the next pointer is correct
Fixes a bug where the next pointer was not being explicitly set. We
were ok much of the time due to zero-filled memory, and also we need
this if the same pdu is re-queued.
Mark Hills [Tue, 7 Jan 2014 10:23:46 +0000 (10:23 +0000)]
Optimisations to the pdu queues
When making many concurrent requests (as is likely in any performance
criticial application), the use of SLIST_REMOVE and SLIST_ADD_END are
a severe bottleneck because of their linear search.
I considered using a double-linked list but it was unnecessary to
allocate the additional memory for each list entry.
Instead, continue to use a single-linked list but retain:
* a pointer to the end of the list; and
* a pointer to the previous entry during a linear search.
The former would makes append operations O(1) time, and the latter
does the same for removal. We can do this because removal only happens
within the linear search, and there is no random access to the queue.
Ronnie Sahlberg [Sun, 23 Mar 2014 16:54:21 +0000 (09:54 -0700)]
RPM: Fix the specfile
Ronnie Sahlberg [Sun, 23 Mar 2014 16:54:05 +0000 (09:54 -0700)]
INCLUDES: Don't install libnfs-provate.h or slist.h
Ronnie Sahlberg [Sun, 23 Mar 2014 16:18:31 +0000 (09:18 -0700)]
AUTOTOOLS: Add Makefile.am to utils
Ronnie Sahlberg [Sun, 23 Mar 2014 16:18:13 +0000 (09:18 -0700)]
SPECFILE: Fix syntax error in the spec file
Ronnie Sahlberg [Sat, 22 Mar 2014 01:31:05 +0000 (18:31 -0700)]
NFS-LS: Build nfs-ls as utils and make it install under bin
Ronnie Sahlberg [Sat, 22 Mar 2014 01:15:40 +0000 (18:15 -0700)]
DOC: Add manpage for nfs-ls
Ronnie Sahlberg [Sat, 22 Mar 2014 00:33:14 +0000 (17:33 -0700)]
examples/Makefile.am Remove stuff we do not need
Ronnie Sahlberg [Fri, 21 Mar 2014 00:32:19 +0000 (17:32 -0700)]
PORTMAP: Add some comments about the origin of definitions
Arne Redlich [Sun, 9 Mar 2014 20:55:42 +0000 (21:55 +0100)]
nfs_normalize_path: don't strip trailing slash from "/"
Otherwise end up with a null string which is not permitted (RFC 1813, 3.2;
the code checks for it right after the now fixed nullification of "/").
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Arne Redlich [Tue, 18 Feb 2014 20:07:46 +0000 (21:07 +0100)]
nfs_opendir2_cb: plug potential memory leak
Spotted by clang analyzer.
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Ronnie Sahlberg [Wed, 19 Mar 2014 03:31:17 +0000 (20:31 -0700)]
PORTMAP client: Pass length as sockaddr_storage, not sockaddr_in6
Ronnie Sahlberg [Wed, 19 Mar 2014 03:27:14 +0000 (20:27 -0700)]
PORTMAP client: Parse and print the actual sockaddr that UADDR2TADDR returned
Ronnie Sahlberg [Wed, 19 Mar 2014 01:59:48 +0000 (18:59 -0700)]
IPV6: Update the README and specify ipv6 is supported
Ronnie Sahlberg [Tue, 18 Mar 2014 13:56:12 +0000 (06:56 -0700)]
PORTMAP client: Add commands to send v3 SET/UNSET
Ronnie Sahlberg [Tue, 18 Mar 2014 04:29:32 +0000 (21:29 -0700)]
PORTMAP: Rename pmap3_getaddr_result to pmap3_string_result
Ronnie Sahlberg [Tue, 18 Mar 2014 04:20:12 +0000 (21:20 -0700)]
PORTMAP: Add support for v3 TADDR2UADDR
Ronnie Sahlberg [Tue, 18 Mar 2014 04:05:18 +0000 (21:05 -0700)]
PORTMAP: Add PMAP v3 UADDR2TADDR support
Ronnie Sahlberg [Tue, 18 Mar 2014 03:19:03 +0000 (20:19 -0700)]
PORTMAP: Add support for PORTMAP v3 CALLIT
Ronnie Sahlberg [Tue, 18 Mar 2014 02:58:48 +0000 (19:58 -0700)]
PORTMAP: Add v3 GETTIME support
Ronnie Sahlberg [Mon, 17 Mar 2014 00:48:35 +0000 (17:48 -0700)]
IPv6: If we use IPv6 then we need to use PMAP v3 GETADDR
Ronnie Sahlberg [Sun, 16 Mar 2014 23:37:33 +0000 (16:37 -0700)]
PORTMAP: Add support for SET UNSET procedures
Ronnie Sahlberg [Sun, 16 Mar 2014 23:28:08 +0000 (16:28 -0700)]
PORTMAP: Add support for PORTMAP v3 GETADDR
Ronnie Sahlberg [Sun, 16 Mar 2014 23:03:19 +0000 (16:03 -0700)]
PORTMAPv3: Add NULL and DUMP commands. Also add portmap example client.
Ronnie Sahlberg [Sun, 16 Mar 2014 22:06:56 +0000 (15:06 -0700)]
GETEXPORT: Use rpc_connect_program_async() flow during list exports
and get rid of several now redundant callbacks.
Ronnie Sahlberg [Sun, 16 Mar 2014 22:00:19 +0000 (15:00 -0700)]
RPC_CONNECT: Use the rpc_connect_program_async() flow during nfs_mount()
Ronnie Sahlberg [Sun, 16 Mar 2014 21:34:32 +0000 (14:34 -0700)]
PORTMAP: Rename the functions in PMAP to PMAP2 to desribe the version of PMAP we use.
Future patches will add PMAP3 which we will need for IPv6 support.
Ronnie Sahlberg [Sun, 16 Mar 2014 16:12:49 +0000 (09:12 -0700)]
IPV6: Add basic IPv6 support
This adds basic IPv6 support to libnfs.
Since libnfs currently only support PORTMAPPER protocol up to version 2
the IPv6 support only works if the server runs Both MOUNT and NFS protocols
on the same ports for IPv6 as for IPv4.
To get full IPv6 support we need to add support for PORTMAPPER version 3
and use it for discovery when using IPv6
Ronnie Sahlberg [Thu, 20 Mar 2014 01:12:45 +0000 (18:12 -0700)]
New version: 1.9.3
Wed Mar 19 2014: Version 1.9.3
- Add O_TRUNC support to nfs_open()
- Add a simple but incomplete LD_PRELOAD tool
- Fixes for some memory leaks and C++ compile support
- Make ANDROID default uid/gid to 65534
- Allow the READDIRPLUS emulation to still work if some objects
in the direcotry can not be lookedup (NFSv4 ACL denying READ-ATTRIBUTES)
- Have libnfs retry any read/write operations where the server responds
with a short read/write. Some servers do this when they are overloaded?
Ronnie Sahlberg [Tue, 18 Mar 2014 19:30:40 +0000 (12:30 -0700)]
Merge pull request #69 from plieven/for-1.9.3
nfs_{pread,pwrite}_async: fix oom handling and comments
Peter Lieven [Tue, 18 Mar 2014 19:08:33 +0000 (20:08 +0100)]
nfs_{pread,pwrite}_async: fix oom handling and comments
Signed-off-by: Peter Lieven <pl@kamp.de>
Ronnie Sahlberg [Mon, 17 Mar 2014 17:29:43 +0000 (10:29 -0700)]
Merge pull request #68 from plieven/for-1.9.3
examples/nfs-ls: fix summary output
Peter Lieven [Mon, 17 Mar 2014 17:05:15 +0000 (18:05 +0100)]
examples/nfs-ls: fix summary output
Signed-off-by: Peter Lieven <pl@kamp.de>
Ronnie Sahlberg [Sun, 16 Mar 2014 21:15:34 +0000 (14:15 -0700)]
PORTMAP: Add support for V3 DUMP command
This implements the missing procedure from Issue #65
Ronnie Sahlberg [Sun, 16 Mar 2014 20:43:04 +0000 (13:43 -0700)]
Merge pull request #67 from plieven/for-1.9.3
For 1.9.3
Peter Lieven [Sat, 15 Mar 2014 16:11:27 +0000 (17:11 +0100)]
{pread,pwrite}_async: fix potential segfault in out of memory condition
if there are already requests in flight we cannot return with an error immediately
from the functions since the caller will likely tidy up his data structures directly
and later on we call his callback with private_data that has likely already
been freed.
Signed-off-by: Peter Lieven <pl@kamp.de>
Peter Lieven [Sun, 16 Mar 2014 19:15:48 +0000 (20:15 +0100)]
nfs_pread_async: handle short reads
the RFC allows the server to read less bytes than requested even
if not at the EOF.
this patch implements a reissue logic for the reminder of the
read request(s).
Signed-off-by: Peter Lieven <pl@kamp.de>
Peter Lieven [Sat, 15 Mar 2014 15:22:03 +0000 (16:22 +0100)]
nfs_pwrite_async: handle short writes
the RFC allows the server to write less bytes than requested.
this patch implements a reissue logic for the reminder of the
write request(s).
Signed-off-by: Peter Lieven <pl@kamp.de>
Peter Lieven [Sat, 15 Mar 2014 14:58:08 +0000 (15:58 +0100)]
fix possible wrong cast to 32-bit unsigned
when calculation the max_offset the (unsigned) leads to a cast
to a 32-bit unsigned integer depending on the platform. as a result
we update the max_offset everytime when it grows beyond 2^32.
this leads to a wrong return max_offset value if the callbacks
are received out of order.
Signed-off-by: Peter Lieven <pl@kamp.de>
Peter Lieven [Sat, 15 Mar 2014 16:05:44 +0000 (17:05 +0100)]
introduce nfs_fill_{READ,WRITE}3args
Signed-off-by: Peter Lieven <pl@kamp.de>
Peter Lieven [Sat, 15 Mar 2014 13:20:29 +0000 (14:20 +0100)]
fix potential overflow in nfs_pread_mcb
Signed-off-by: Peter Lieven <pl@kamp.de>
Ronnie Sahlberg [Thu, 13 Mar 2014 13:30:31 +0000 (06:30 -0700)]
Improve error messages when READDIRPLUS emulation fails.
This addresses issue #62
Ronnie Sahlberg [Thu, 13 Mar 2014 04:00:08 +0000 (21:00 -0700)]
Dont check for NFS errors in the LOOKUP during READDIRPLUS emulation
On servers with extended attributes, a server copuld be set up to
deny READ-ATTRIBUTES for the libnfs user.
This means that READDIRPLUS will no longer work since it will need to
stat() and thus READ-ATTRIBUTE in order to prepare the response.
Libnfs has READDIRPLUS emulation for the cases where this command fails
by switching to old READDIR to scan all the file names and then a LOOKUP loop for getting the file attributes.
Most of the time the purpose for this emulation is to handle the case where the server simply does not support READDIRPLUS at all, which sometimes is the case for embedded systems with userspace nfs servers.
In this case, where files just have READ-ATTRIBUTE deny for the libnfs user,
this will also fail and trigger the fallback to READDIR + LOOKUP-loop.
If the LOOKUP fails for this loop, then just ignore trying to update the attributes we have for this object, but do not fail the actual READDIRPLUS emulation.
This addresses a permissions issue reported by a XBMC user in issue #60
Ronnie Sahlberg [Thu, 13 Mar 2014 02:31:20 +0000 (19:31 -0700)]
Set default uid/gid to 65534/nobody for ANDROID
Ronnie Sahlberg [Wed, 12 Mar 2014 12:54:38 +0000 (05:54 -0700)]
We only need to spend CPU computing a new error string IFF there was an error
when unmarshalling the reply.
Ronnie Sahlberg [Wed, 12 Mar 2014 01:52:48 +0000 (18:52 -0700)]
Improve error reporting when unmarshalling of message headers fail.
Ronnie Sahlberg [Wed, 5 Mar 2014 03:40:05 +0000 (19:40 -0800)]
Add a new nfs_stat64() function that operates on a always-64-bit stat structure
Arne Redlich [Tue, 18 Feb 2014 19:56:24 +0000 (20:56 +0100)]
nfs_pread_async: plug potential memleaks
Spotted by clang analyzer.
This also introduces asserts to help clang analyzer avoid reporting
false positives.
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Arne Redlich [Tue, 18 Feb 2014 19:51:36 +0000 (20:51 +0100)]
nfs_write_async: plug potential memory leaks
Spotted by clang analyzer.
This also introduces asserts to help clang analyzer avoid
reporting false positives.
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Arne Redlich [Tue, 18 Feb 2014 19:22:11 +0000 (20:22 +0100)]
rpc_read_from_socket: fix use-after-free due to missing return
Spotted by clang analyzer.
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Arne Redlich [Mon, 17 Feb 2014 23:42:14 +0000 (00:42 +0100)]
socket.c: fix format string issues (too few arguments)
These were uncovered by the previously added __attribute__((format(printf))).
Emacs also removed trailing whitespace while at it.
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Arne Redlich [Mon, 17 Feb 2014 23:41:05 +0000 (00:41 +0100)]
{nfs,rpc}_set_error: add __attribute__((format(printf))) annotations
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Arne Redlich [Mon, 17 Feb 2014 23:32:10 +0000 (00:32 +0100)]
nfs_rename_continue_1_internal: fix use-after-free
Spotted by clang analyzer.
This introduces another allocation to create a copy of the target path
of a rename in case it needs to be reported via rpc_set_error - it might
be a better idea to avoid the allocation and have a slightly less informative
error message?
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>
Arne Redlich [Mon, 17 Feb 2014 23:22:45 +0000 (00:22 +0100)]
nfs_unlink_async: plug potential memory leak
Spotted by clang analyzer
Signed-off-by: Arne Redlich <arne.redlich@googlemail.com>