Return EINVAL when trying to use CURVE without libsodium

src/options.cpp

@@ -284,12 +284,10 @@ int zmq::options_t::setsockopt (int option_, const void *optval_,
                 return 0;
             }
             break;
-            
+        
+//  If libsodium isn't installed, these options provoke EINVAL
+#       ifdef HAVE_LIBSODIUM
         case ZMQ_CURVE_SERVER:
-#           ifndef HAVE_LIBSODIUM
-            puts ("E: libzmq was not built using libsodium, CURVE not available");
-            assert (false);
-#           endif
             if (is_int && (value == 0 || value == 1)) {
                 as_server = value;
                 mechanism = value? ZMQ_CURVE: ZMQ_NULL;
@@ -298,10 +296,6 @@ int zmq::options_t::setsockopt (int option_, const void *optval_,
             break;
             
         case ZMQ_CURVE_PUBLICKEY:
-#           ifndef HAVE_LIBSODIUM
-            puts ("E: libzmq was not built using libsodium, CURVE not available");
-            assert (false);
-#           endif
             if (optvallen_ == CURVE_KEYSIZE) {
                 memcpy (curve_public_key, optval_, CURVE_KEYSIZE);
                 mechanism = ZMQ_CURVE;
@@ -310,10 +304,6 @@ int zmq::options_t::setsockopt (int option_, const void *optval_,
             break;
             
         case ZMQ_CURVE_SECRETKEY:
-#           ifndef HAVE_LIBSODIUM
-            puts ("E: libzmq was not built using libsodium, CURVE not available");
-            assert (false);
-#           endif
             if (optvallen_ == CURVE_KEYSIZE) {
                 memcpy (curve_secret_key, optval_, CURVE_KEYSIZE);
                 mechanism = ZMQ_CURVE;
@@ -322,10 +312,6 @@ int zmq::options_t::setsockopt (int option_, const void *optval_,
             break;
             
         case ZMQ_CURVE_SERVERKEY:
-#           ifndef HAVE_LIBSODIUM
-            puts ("E: libzmq was not built using libsodium, CURVE not available");
-            assert (false);
-#           endif
             if (optvallen_ == CURVE_KEYSIZE) {
                 memcpy (curve_server_key, optval_, CURVE_KEYSIZE);
                 as_server = 0;
@@ -333,9 +319,7 @@ int zmq::options_t::setsockopt (int option_, const void *optval_,
                 return 0;
             }
             break;
-            
-        default:
-            break;
+#       endif
     }
     errno = EINVAL;
     return -1;
@@ -555,6 +539,8 @@ int zmq::options_t::getsockopt (int option_, void *optval_, size_t *optvallen_)
             }
             break;
             
+//  If libsodium isn't installed, these options provoke EINVAL
+#       ifdef HAVE_LIBSODIUM
         case ZMQ_CURVE_SERVER:
             if (is_int) {
                 *value = as_server && mechanism == ZMQ_CURVE;
@@ -582,6 +568,7 @@ int zmq::options_t::getsockopt (int option_, void *optval_, size_t *optvallen_)
                 return 0;
             }
             break;
+#       endif
     }
     errno = EINVAL;
     return -1;