raise BufferError consistently in cffi backend

Author: minrk

zmq/backend/cffi/message.py

@@ -82,6 +82,9 @@ def __init__(self, data=None, track=False, copy=None, copy_threshold=None):
             self._bytes = data
 
         self._buffer = memoryview(data)
+        if not self._buffer.contiguous:
+            raise BufferError("memoryview: underlying buffer is not contiguous")
+        # from_buffer silently copies if memory is not contiguous
         c_data = ffi.from_buffer(self._buffer)
         data_len_c = self._buffer.nbytes
 

zmq/backend/cffi/socket.py

@@ -338,9 +338,9 @@ def recv(self, flags=0, copy=True, track=False):
     def recv_into(self, buffer, /, *, nbytes: int = 0, flags: int = 0) -> int:
         view = memoryview(buffer)
         if not view.contiguous:
-            raise ValueError("Can only recv_into contiguous buffers")
+            raise BufferError("Can only recv_into contiguous buffers")
         if view.readonly:
-            raise ValueError("Cannot recv_into readonly buffer")
+            raise BufferError("Cannot recv_into readonly buffer")
         if nbytes < 0:
             raise ValueError(f"{nbytes=} must be non-negative")
         view_bytes = view.nbytes