[libvirt] [PATCH 1/2] virnetserverservice: Resolve Coverity ARRAY_VS_SINGLETON

John Ferlan jferlan at redhat.com
Thu Aug 28 19:28:47 UTC 2014


Coverity complained about the following:

(3) Event ptr_arith:
   Performing pointer arithmetic on "cur_fd" in expression "cur_fd++".
130             return virNetServerServiceNewFD(*cur_fd++,

It seems the belief is their is pointer arithmetic taking place.  By using
(*cur_fd)++ we avoid this possible issue

Signed-off-by: John Ferlan <jferlan at redhat.com>
---
 src/rpc/virnetserverservice.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/rpc/virnetserverservice.c b/src/rpc/virnetserverservice.c
index fea05c3..486f93e 100644
--- a/src/rpc/virnetserverservice.c
+++ b/src/rpc/virnetserverservice.c
@@ -127,7 +127,7 @@ virNetServerServiceNewFDOrUNIX(const char *path,
          * There's still enough file descriptors.  In this case we'll
          * use the current one and increment it afterwards.
          */
-        return virNetServerServiceNewFD(*cur_fd++,
+        return virNetServerServiceNewFD((*cur_fd)++,
                                         auth,
 #if WITH_GNUTLS
                                         tls,
-- 
1.9.3




More information about the libvir-list mailing list