fix compiler flags and warnings

diff --git a/Makefile b/Makefile
index 5cd0ee90ca6de9a01727335432dd85e625b33d7c..550d0fce73dcd5a1509b63adf4c2965e915056b1 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -27,6 +27,9 @@ RADOS.c: RADOS.xs typemap
        mv RADOS.xsc RADOS.c
 
 CFLAGS= -shared -fPIC -O2 -Wall -Wl,-z,relro -I/usr/lib/perl/5.14.2/CORE -DXS_VERSION=\"1.0\"
+CFLAGS= -shared -fPIC -O2 -Werror -Wtype-limits -Wall -Wl,-z,relro \
+       -D_FORTIFY_SOURCE=2 -I/usr/lib/perl/5.14.2/CORE -DXS_VERSION=\"1.0\"
+
 RADOS.so: RADOS.c
        gcc ${CFLAGS} -lrados -o RADOS.so RADOS.c
 

diff --git a/RADOS.xs b/RADOS.xs
index beef1a4c1ff5979de444b01e6b948e59c669b4e4..fa472c4afdf03f76618e11f9d35fd6f87aa664c5 100644 (file)
--- a/RADOS.xs
+++ b/RADOS.xs
@@ -119,12 +119,11 @@ CODE:
         XSRETURN_UNDEF;
     }
     HV * rh = (HV *)sv_2mortal((SV *)newHV());
-    SV **t;
 
-    t = hv_store(rh, "kb", 2, newSViv(result.kb), 0);
-    t = hv_store(rh, "kb_used", 7, newSViv(result.kb_used), 0);
-    t = hv_store(rh, "kb_avail", 8, newSViv(result.kb_avail), 0);
-    t = hv_store(rh, "num_objects", 11, newSViv(result.num_objects), 0);
+    (void)hv_store(rh, "kb", 2, newSViv(result.kb), 0);
+    (void)hv_store(rh, "kb_used", 7, newSViv(result.kb_used), 0);
+    (void)hv_store(rh, "kb_avail", 8, newSViv(result.kb_avail), 0);
+    (void)hv_store(rh, "num_objects", 11, newSViv(result.num_objects), 0);
 
     RETVAL = rh;
 }