[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #4135: net/rpmsg: add multi-iov support

[GitBox <gi...@apache.org>]

gustavonihei commented on a change in pull request #4135:
URL: https://github.com/apache/incubator-nuttx/pull/4135#discussion_r671443436



##########
File path: net/rpmsg/rpmsg_sockif.c
##########
@@ -991,7 +1021,7 @@ static ssize_t rpmsg_socket_send_single(FAR struct socket *psock,
   if (total > ipcsize)
     {
       total = ipcsize;
-      len   = ipcsize - sizeof(*msg) + sizeof(uint32_t);
+      len = ipcsize - sizeof(*msg) - sizeof(uint32_t);

Review comment:
       This seems to be a bugfix that is not strictly related to the new feature being added in this PR.
   If so, please add it to a dedicated commit.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org