diff options
| author |   Florin Coras <fcoras@cisco.com> | 2021-06-04 17:31:53 -0700 |
|---|---|---|
| committer |   Dave Barach <openvpp@barachs.net> | 2021-06-07 15:24:40 +0000 |
| commit | 0b0d28e9bf1ea078ce1a846d22e499d13eadd464 (patch) | |
| tree | 967f164dda49f3204152d222d5456820ee24d399 | |
| parent | 233683180638d1c7f517b713722cbbeb3b4be86e (diff) | |
vcl: accept zero length writes
Type: fix
Signed-off-by: Florin Coras <fcoras@cisco.com>
Change-Id: I84985f59274e77219a38ea6ac865fc73ac635d72
| -rw-r--r-- | src/vcl/vppcom.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/src/vcl/vppcom.c b/src/vcl/vppcom.c index 4c77b0bc43b..0c6f6e6bbfd 100644 --- a/src/vcl/vppcom.c +++ b/src/vcl/vppcom.c @@ -2152,8 +2152,12 @@ vppcom_session_write_inline (vcl_worker_t * wrk, vcl_session_t * s, void *buf, svm_msg_q_t *mq; u8 is_ct; - if (PREDICT_FALSE (!buf || n == 0)) - return VPPCOM_EINVAL; + /* Accept zero length writes but just return */ + if (PREDICT_FALSE (!n)) + return VPPCOM_OK; + + if (PREDICT_FALSE (!buf)) + return VPPCOM_EFAULT; if (PREDICT_FALSE (s->flags & VCL_SESSION_F_IS_VEP)) { @@ -2163,7 +2167,7 @@ vppcom_session_write_inline (vcl_worker_t * wrk, vcl_session_t * s, void *buf, } if (PREDICT_FALSE (!vcl_session_is_open (s) || - s->flags & VCL_SESSION_F_SHUTDOWN)) + (s->flags & VCL_SESSION_F_SHUTDOWN))) { VDBG (1, "session %u [0x%llx]: is not open! state 0x%x (%s)", s->session_index, s->vpp_handle, s->session_state, @@ -3889,12 +3893,9 @@ vppcom_session_sendto (uint32_t session_handle, void *buffer, vcl_session_t *s; s = vcl_session_get_w_handle (wrk, session_handle); - if (!s) + if (PREDICT_FALSE (!s)) return VPPCOM_EBADFD; - if (!buffer) - return VPPCOM_EINVAL; - if (ep) { if (!vcl_session_is_cl (s)) |