nuttx/net/usrsock/usrsock_recvmsg.c

450 lines
12 KiB
C
Raw Normal View History

/****************************************************************************
* net/usrsock/usrsock_recvmsg.c
*
* Licensed to the Apache Software Foundation (ASF) under one or more
* contributor license agreements. See the NOTICE file distributed with
* this work for additional information regarding copyright ownership. The
* ASF licenses this file to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance with the
* License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
* WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
* License for the specific language governing permissions and limitations
* under the License.
*
****************************************************************************/
/****************************************************************************
* Included Files
****************************************************************************/
#include <nuttx/config.h>
#if defined(CONFIG_NET) && defined(CONFIG_NET_USRSOCK)
#include <stdint.h>
#include <string.h>
#include <assert.h>
#include <errno.h>
#include <debug.h>
#include <arch/irq.h>
#include <sys/socket.h>
#include <nuttx/net/net.h>
#include <nuttx/net/usrsock.h>
#include "usrsock/usrsock.h"
/****************************************************************************
* Private Functions
****************************************************************************/
static uint16_t recvfrom_event(FAR struct net_driver_s *dev,
FAR void *pvconn, FAR void *pvpriv,
uint16_t flags)
{
FAR struct usrsock_data_reqstate_s *pstate = pvpriv;
FAR struct usrsock_conn_s *conn = pvconn;
if (flags & USRSOCK_EVENT_ABORT)
{
ninfo("socket aborted.\n");
pstate->reqstate.result = -ECONNABORTED;
pstate->valuelen = 0;
pstate->valuelen_nontrunc = 0;
/* Stop further callbacks */
pstate->reqstate.cb->flags = 0;
pstate->reqstate.cb->priv = NULL;
pstate->reqstate.cb->event = NULL;
/* Wake up the waiting thread */
nxsem_post(&pstate->reqstate.recvsem);
}
else if (flags & USRSOCK_EVENT_REQ_COMPLETE)
{
ninfo("request completed.\n");
pstate->reqstate.result = conn->resp.result;
if (pstate->reqstate.result < 0)
{
pstate->valuelen = 0;
pstate->valuelen_nontrunc = 0;
}
else
{
pstate->valuelen = conn->resp.valuelen;
pstate->valuelen_nontrunc = conn->resp.valuelen_nontrunc;
}
if (pstate->reqstate.result >= 0 ||
pstate->reqstate.result == -EAGAIN)
{
/* After reception of data, mark input not ready. Daemon will
* send event to restore this flag.
*/
conn->flags &= ~USRSOCK_EVENT_RECVFROM_AVAIL;
}
/* Stop further callbacks */
pstate->reqstate.cb->flags = 0;
pstate->reqstate.cb->priv = NULL;
pstate->reqstate.cb->event = NULL;
/* Wake up the waiting thread */
nxsem_post(&pstate->reqstate.recvsem);
}
else if (flags & USRSOCK_EVENT_RECVFROM_AVAIL)
{
ninfo("recvfrom avail.\n");
flags &= ~USRSOCK_EVENT_RECVFROM_AVAIL;
/* Stop further callbacks */
pstate->reqstate.cb->flags = 0;
pstate->reqstate.cb->priv = NULL;
pstate->reqstate.cb->event = NULL;
/* Wake up the waiting thread */
nxsem_post(&pstate->reqstate.recvsem);
}
else if (flags & USRSOCK_EVENT_REMOTE_CLOSED)
{
ninfo("remote closed.\n");
pstate->reqstate.result = -EPIPE;
/* Stop further callbacks */
pstate->reqstate.cb->flags = 0;
pstate->reqstate.cb->priv = NULL;
pstate->reqstate.cb->event = NULL;
/* Wake up the waiting thread */
nxsem_post(&pstate->reqstate.recvsem);
}
return flags;
}
/****************************************************************************
* Name: do_recvfrom_request
****************************************************************************/
static int do_recvfrom_request(FAR struct usrsock_conn_s *conn,
size_t buflen, socklen_t addrlen,
int32_t flags)
{
struct usrsock_request_recvfrom_s req =
{
};
2019-10-25 19:31:42 +02:00
struct iovec bufs[1];
if (addrlen > UINT16_MAX)
{
addrlen = UINT16_MAX;
}
if (buflen > UINT32_MAX)
{
buflen = UINT32_MAX;
}
/* Prepare request for daemon to read. */
req.head.reqid = USRSOCK_REQUEST_RECVFROM;
req.usockid = conn->usockid;
req.flags = flags;
req.max_addrlen = addrlen;
req.max_buflen = buflen;
bufs[0].iov_base = (FAR void *)&req;
bufs[0].iov_len = sizeof(req);
return usrsockdev_do_request(conn, bufs, ARRAY_SIZE(bufs));
}
/****************************************************************************
* Name: usrsock_recvmsg
*
* Description:
* recvmsg() receives messages from a socket, and may be used to receive
* data on a socket whether or not it is connection-oriented.
*
* If from is not NULL, and the underlying protocol provides the source
* address, this source address is filled in. The argument fromlen
* initialized to the size of the buffer associated with from, and modified
* on return to indicate the actual size of the address stored there.
*
* Input Parameters:
* psock A pointer to a NuttX-specific, internal socket structure
* msg Buffer to receive the message
* flags Receive flags (ignored)
*
* Returned Value:
* On success, returns the number of characters received. If no data is
* available to be received and the peer has performed an orderly shutdown,
* recvfrom() will return 0. Otherwise, on any failure, a negated errno
* value is returned.
*
****************************************************************************/
ssize_t usrsock_recvmsg(FAR struct socket *psock, FAR struct msghdr *msg,
int flags)
{
FAR void *buf = msg->msg_iov->iov_base;
size_t len = msg->msg_iov->iov_len;
FAR struct sockaddr *from = msg->msg_name;
FAR socklen_t *fromlen = &msg->msg_namelen;
FAR struct usrsock_conn_s *conn = psock->s_conn;
struct usrsock_data_reqstate_s state =
{
};
2019-10-25 19:31:42 +02:00
struct iovec inbufs[2];
socklen_t addrlen = 0;
socklen_t outaddrlen = 0;
ssize_t ret;
DEBUGASSERT(conn);
if (fromlen)
{
if (*fromlen > 0 && from == NULL)
{
return -EINVAL;
}
addrlen = *fromlen;
}
net_lock();
if (conn->state == USRSOCK_CONN_STATE_UNINITIALIZED ||
conn->state == USRSOCK_CONN_STATE_ABORTED)
{
/* Invalid state or closed by daemon. */
ninfo("usockid=%d; recvfrom() with uninitialized usrsock.\n",
conn->usockid);
ret = (conn->state == USRSOCK_CONN_STATE_ABORTED) ? -EPIPE :
-ECONNRESET;
goto errout_unlock;
}
if (conn->type == SOCK_STREAM || conn->type == SOCK_SEQPACKET)
{
if (!conn->connected)
{
if (conn->state == USRSOCK_CONN_STATE_CONNECTING)
{
/* Connecting. */
ninfo("usockid=%d; socket still connecting.\n",
conn->usockid);
ret = -EAGAIN;
goto errout_unlock;
}
else
{
/* Not connected. */
ninfo("usockid=%d; socket not connected.\n",
conn->usockid);
ret = -ENOTCONN;
goto errout_unlock;
}
}
}
if (conn->state == USRSOCK_CONN_STATE_CONNECTING)
{
/* Non-blocking connecting. */
ninfo("usockid=%d; socket still connecting.\n",
conn->usockid);
ret = -EAGAIN;
goto errout_unlock;
}
do
{
/* Check if remote end has closed connection. */
if (conn->flags & USRSOCK_EVENT_REMOTE_CLOSED &&
!(conn->flags & USRSOCK_EVENT_RECVFROM_AVAIL))
{
ninfo("usockid=%d; remote closed (EOF).\n", conn->usockid);
ret = 0;
goto errout_unlock;
}
/* Check if need to wait for receive data to become available. */
if (!(conn->flags & USRSOCK_EVENT_RECVFROM_AVAIL))
{
if (_SS_ISNONBLOCK(conn->sconn.s_flags) ||
(flags & MSG_DONTWAIT) != 0)
{
/* Nothing to receive from daemon side. */
ret = -EAGAIN;
goto errout_unlock;
}
/* Wait recv to become avail. */
ret = usrsock_setup_data_request_callback(
conn, &state, recvfrom_event,
USRSOCK_EVENT_ABORT | USRSOCK_EVENT_RECVFROM_AVAIL |
USRSOCK_EVENT_REMOTE_CLOSED);
if (ret < 0)
{
net/usrsock: Fix the compile warning In file included from usrsock/usrsock_bind.c:32: usrsock/usrsock_bind.c: In function ‘usrsock_bind’: usrsock/usrsock_bind.c:183:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 183 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_bind.c:183:54: note: format string is defined here 183 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_connect.c CC: usrsock/usrsock_dev.c In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_handle_event’: usrsock/usrsock_dev.c:488:19: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:488:40: note: format string is defined here 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c:488:19: warning: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘long unsigned int’ [-Wformat=] 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~ | | | long unsigned int usrsock/usrsock_dev.c:488:45: note: format string is defined here 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_handle_datareq_response’: usrsock/usrsock_dev.c:657:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 657 | nwarn("%dth buffer not large enough (need: %d, have: %d).\n", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ...... 660 | conn->resp.datain.iov[iovpos].iov_len); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:657:61: note: format string is defined here 657 | nwarn("%dth buffer not large enough (need: %d, have: %d).\n", | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c:678:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 678 | nwarn("%dth buffer not large enough " | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ...... 682 | conn->resp.datain.iov[iovpos].iov_len); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:679:45: note: format string is defined here 679 | "(need: %" PRId32 ", have: %d).\n", | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_handle_req_response’: usrsock/usrsock_dev.c:745:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 745 | nwarn("message too short, %d < %d.\n", len, hdrlen); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:745:34: note: format string is defined here 745 | nwarn("message too short, %d < %d.\n", len, hdrlen); | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_write’: usrsock/usrsock_dev.c:858:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 858 | nwarn("message too short, %d < %d.\n", len, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:858:38: note: format string is defined here 858 | nwarn("message too short, %d < %d.\n", len, | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c:858:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘long unsigned int’ [-Wformat=] 858 | nwarn("message too short, %d < %d.\n", len, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 859 | sizeof(struct usrsock_message_common_s)); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | | | long unsigned int usrsock/usrsock_dev.c:858:43: note: format string is defined here 858 | nwarn("message too short, %d < %d.\n", len, | ~^ | | | int | %ld CC: usrsock/usrsock_getpeername.c In file included from usrsock/usrsock_getpeername.c:32: usrsock/usrsock_getpeername.c: In function ‘usrsock_getpeername’: usrsock/usrsock_getpeername.c:190:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_getpeername.c:190:54: note: format string is defined here 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_event.c CC: usrsock/usrsock_getsockname.c In file included from usrsock/usrsock_getsockname.c:32: usrsock/usrsock_getsockname.c: In function ‘usrsock_getsockname’: usrsock/usrsock_getsockname.c:190:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_getsockname.c:190:54: note: format string is defined here 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_getsockopt.c CC: usrsock/usrsock_poll.c CC: usrsock/usrsock_recvmsg.c In file included from usrsock/usrsock_recvmsg.c:32: usrsock/usrsock_recvmsg.c: In function ‘usrsock_recvmsg’: usrsock/usrsock_recvmsg.c:321:21: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 321 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_recvmsg.c:321:62: note: format string is defined here 321 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_recvmsg.c:32: usrsock/usrsock_recvmsg.c:343:24: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 343 | nerr("net_timedwait errno: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_recvmsg.c:343:47: note: format string is defined here 343 | nerr("net_timedwait errno: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_recvmsg.c:32: usrsock/usrsock_recvmsg.c:384:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 384 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_recvmsg.c:384:58: note: format string is defined here 384 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_sendmsg.c In file included from usrsock/usrsock_sendmsg.c:32: usrsock/usrsock_sendmsg.c: In function ‘usrsock_sendmsg’: usrsock/usrsock_sendmsg.c:302:21: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 302 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_sendmsg.c:302:62: note: format string is defined here 302 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_sendmsg.c:32: usrsock/usrsock_sendmsg.c:324:24: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 324 | nerr("net_timedwait errno: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_sendmsg.c:324:47: note: format string is defined here 324 | nerr("net_timedwait errno: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_sendmsg.c:32: usrsock/usrsock_sendmsg.c:364:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 364 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_sendmsg.c:364:58: note: format string is defined here 364 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com>
2021-12-31 14:52:10 +01:00
nwarn("usrsock_setup_request_callback failed: %zd\n", ret);
goto errout_unlock;
}
/* Wait for receive-avail (or abort, or timeout, or signal). */
ret = net_timedwait(&state.reqstate.recvsem,
_SO_TIMEOUT(conn->sconn.s_rcvtimeo));
if (ret < 0)
{
if (ret == -ETIMEDOUT)
{
ninfo("recvfrom timedout\n");
ret = -EAGAIN;
}
else if (ret == -EINTR)
{
ninfo("recvfrom interrupted\n");
}
else
{
net/usrsock: Fix the compile warning In file included from usrsock/usrsock_bind.c:32: usrsock/usrsock_bind.c: In function ‘usrsock_bind’: usrsock/usrsock_bind.c:183:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 183 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_bind.c:183:54: note: format string is defined here 183 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_connect.c CC: usrsock/usrsock_dev.c In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_handle_event’: usrsock/usrsock_dev.c:488:19: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:488:40: note: format string is defined here 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c:488:19: warning: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘long unsigned int’ [-Wformat=] 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~ | | | long unsigned int usrsock/usrsock_dev.c:488:45: note: format string is defined here 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_handle_datareq_response’: usrsock/usrsock_dev.c:657:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 657 | nwarn("%dth buffer not large enough (need: %d, have: %d).\n", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ...... 660 | conn->resp.datain.iov[iovpos].iov_len); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:657:61: note: format string is defined here 657 | nwarn("%dth buffer not large enough (need: %d, have: %d).\n", | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c:678:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 678 | nwarn("%dth buffer not large enough " | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ...... 682 | conn->resp.datain.iov[iovpos].iov_len); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:679:45: note: format string is defined here 679 | "(need: %" PRId32 ", have: %d).\n", | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_handle_req_response’: usrsock/usrsock_dev.c:745:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 745 | nwarn("message too short, %d < %d.\n", len, hdrlen); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:745:34: note: format string is defined here 745 | nwarn("message too short, %d < %d.\n", len, hdrlen); | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_write’: usrsock/usrsock_dev.c:858:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 858 | nwarn("message too short, %d < %d.\n", len, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:858:38: note: format string is defined here 858 | nwarn("message too short, %d < %d.\n", len, | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c:858:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘long unsigned int’ [-Wformat=] 858 | nwarn("message too short, %d < %d.\n", len, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 859 | sizeof(struct usrsock_message_common_s)); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | | | long unsigned int usrsock/usrsock_dev.c:858:43: note: format string is defined here 858 | nwarn("message too short, %d < %d.\n", len, | ~^ | | | int | %ld CC: usrsock/usrsock_getpeername.c In file included from usrsock/usrsock_getpeername.c:32: usrsock/usrsock_getpeername.c: In function ‘usrsock_getpeername’: usrsock/usrsock_getpeername.c:190:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_getpeername.c:190:54: note: format string is defined here 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_event.c CC: usrsock/usrsock_getsockname.c In file included from usrsock/usrsock_getsockname.c:32: usrsock/usrsock_getsockname.c: In function ‘usrsock_getsockname’: usrsock/usrsock_getsockname.c:190:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_getsockname.c:190:54: note: format string is defined here 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_getsockopt.c CC: usrsock/usrsock_poll.c CC: usrsock/usrsock_recvmsg.c In file included from usrsock/usrsock_recvmsg.c:32: usrsock/usrsock_recvmsg.c: In function ‘usrsock_recvmsg’: usrsock/usrsock_recvmsg.c:321:21: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 321 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_recvmsg.c:321:62: note: format string is defined here 321 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_recvmsg.c:32: usrsock/usrsock_recvmsg.c:343:24: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 343 | nerr("net_timedwait errno: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_recvmsg.c:343:47: note: format string is defined here 343 | nerr("net_timedwait errno: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_recvmsg.c:32: usrsock/usrsock_recvmsg.c:384:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 384 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_recvmsg.c:384:58: note: format string is defined here 384 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_sendmsg.c In file included from usrsock/usrsock_sendmsg.c:32: usrsock/usrsock_sendmsg.c: In function ‘usrsock_sendmsg’: usrsock/usrsock_sendmsg.c:302:21: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 302 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_sendmsg.c:302:62: note: format string is defined here 302 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_sendmsg.c:32: usrsock/usrsock_sendmsg.c:324:24: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 324 | nerr("net_timedwait errno: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_sendmsg.c:324:47: note: format string is defined here 324 | nerr("net_timedwait errno: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_sendmsg.c:32: usrsock/usrsock_sendmsg.c:364:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 364 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_sendmsg.c:364:58: note: format string is defined here 364 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com>
2021-12-31 14:52:10 +01:00
nerr("net_timedwait errno: %zd\n", ret);
DEBUGPANIC();
}
}
usrsock_teardown_data_request_callback(&state);
/* Did wait timeout or got signal? */
if (ret != 0)
{
goto errout_unlock;
}
/* Was socket aborted? */
if (conn->state == USRSOCK_CONN_STATE_ABORTED)
{
ret = -EPIPE;
goto errout_unlock;
}
/* Did remote disconnect? */
if (conn->flags & USRSOCK_EVENT_REMOTE_CLOSED &&
!(conn->flags & USRSOCK_EVENT_RECVFROM_AVAIL))
{
ret = 0;
goto errout_unlock;
}
DEBUGASSERT(conn->flags & USRSOCK_EVENT_RECVFROM_AVAIL);
}
/* Set up event callback for usrsock. */
ret = usrsock_setup_data_request_callback(
conn, &state, recvfrom_event,
USRSOCK_EVENT_ABORT | USRSOCK_EVENT_REQ_COMPLETE);
if (ret < 0)
{
net/usrsock: Fix the compile warning In file included from usrsock/usrsock_bind.c:32: usrsock/usrsock_bind.c: In function ‘usrsock_bind’: usrsock/usrsock_bind.c:183:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 183 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_bind.c:183:54: note: format string is defined here 183 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_connect.c CC: usrsock/usrsock_dev.c In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_handle_event’: usrsock/usrsock_dev.c:488:19: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:488:40: note: format string is defined here 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c:488:19: warning: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘long unsigned int’ [-Wformat=] 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~ | | | long unsigned int usrsock/usrsock_dev.c:488:45: note: format string is defined here 488 | nwarn("message too short, %d < %d.\n", len, sizeof(*hdr)); | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_handle_datareq_response’: usrsock/usrsock_dev.c:657:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 657 | nwarn("%dth buffer not large enough (need: %d, have: %d).\n", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ...... 660 | conn->resp.datain.iov[iovpos].iov_len); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:657:61: note: format string is defined here 657 | nwarn("%dth buffer not large enough (need: %d, have: %d).\n", | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c:678:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 678 | nwarn("%dth buffer not large enough " | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ...... 682 | conn->resp.datain.iov[iovpos].iov_len); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:679:45: note: format string is defined here 679 | "(need: %" PRId32 ", have: %d).\n", | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_handle_req_response’: usrsock/usrsock_dev.c:745:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 745 | nwarn("message too short, %d < %d.\n", len, hdrlen); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:745:34: note: format string is defined here 745 | nwarn("message too short, %d < %d.\n", len, hdrlen); | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c: In function ‘usrsockdev_write’: usrsock/usrsock_dev.c:858:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=] 858 | nwarn("message too short, %d < %d.\n", len, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | size_t {aka long unsigned int} usrsock/usrsock_dev.c:858:38: note: format string is defined here 858 | nwarn("message too short, %d < %d.\n", len, | ~^ | | | int | %ld In file included from usrsock/usrsock_dev.c:37: usrsock/usrsock_dev.c:858:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘long unsigned int’ [-Wformat=] 858 | nwarn("message too short, %d < %d.\n", len, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 859 | sizeof(struct usrsock_message_common_s)); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | | | long unsigned int usrsock/usrsock_dev.c:858:43: note: format string is defined here 858 | nwarn("message too short, %d < %d.\n", len, | ~^ | | | int | %ld CC: usrsock/usrsock_getpeername.c In file included from usrsock/usrsock_getpeername.c:32: usrsock/usrsock_getpeername.c: In function ‘usrsock_getpeername’: usrsock/usrsock_getpeername.c:190:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_getpeername.c:190:54: note: format string is defined here 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_event.c CC: usrsock/usrsock_getsockname.c In file included from usrsock/usrsock_getsockname.c:32: usrsock/usrsock_getsockname.c: In function ‘usrsock_getsockname’: usrsock/usrsock_getsockname.c:190:13: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_getsockname.c:190:54: note: format string is defined here 190 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_getsockopt.c CC: usrsock/usrsock_poll.c CC: usrsock/usrsock_recvmsg.c In file included from usrsock/usrsock_recvmsg.c:32: usrsock/usrsock_recvmsg.c: In function ‘usrsock_recvmsg’: usrsock/usrsock_recvmsg.c:321:21: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 321 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_recvmsg.c:321:62: note: format string is defined here 321 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_recvmsg.c:32: usrsock/usrsock_recvmsg.c:343:24: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 343 | nerr("net_timedwait errno: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_recvmsg.c:343:47: note: format string is defined here 343 | nerr("net_timedwait errno: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_recvmsg.c:32: usrsock/usrsock_recvmsg.c:384:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 384 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_recvmsg.c:384:58: note: format string is defined here 384 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld CC: usrsock/usrsock_sendmsg.c In file included from usrsock/usrsock_sendmsg.c:32: usrsock/usrsock_sendmsg.c: In function ‘usrsock_sendmsg’: usrsock/usrsock_sendmsg.c:302:21: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 302 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_sendmsg.c:302:62: note: format string is defined here 302 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_sendmsg.c:32: usrsock/usrsock_sendmsg.c:324:24: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 324 | nerr("net_timedwait errno: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_sendmsg.c:324:47: note: format string is defined here 324 | nerr("net_timedwait errno: %d\n", ret); | ~^ | | | int | %ld In file included from usrsock/usrsock_sendmsg.c:32: usrsock/usrsock_sendmsg.c:364:17: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ {aka ‘long int’} [-Wformat=] 364 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~ | | | ssize_t {aka long int} usrsock/usrsock_sendmsg.c:364:58: note: format string is defined here 364 | nwarn("usrsock_setup_request_callback failed: %d\n", ret); | ~^ | | | int | %ld Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com>
2021-12-31 14:52:10 +01:00
nwarn("usrsock_setup_request_callback failed: %zd\n", ret);
goto errout_unlock;
}
inbufs[0].iov_base = (FAR void *)from;
inbufs[0].iov_len = addrlen;
inbufs[1].iov_base = (FAR void *)buf;
inbufs[1].iov_len = len;
usrsock_setup_datain(conn, inbufs, ARRAY_SIZE(inbufs));
/* MSG_DONTWAIT is only use in usrsock. */
flags &= ~MSG_DONTWAIT;
/* Request user-space daemon to close socket. */
ret = do_recvfrom_request(conn, len, addrlen, flags);
if (ret >= 0)
{
/* Wait for completion of request. */
net_lockedwait_uninterruptible(&state.reqstate.recvsem);
ret = state.reqstate.result;
DEBUGASSERT(ret <= (ssize_t)len);
DEBUGASSERT(state.valuelen <= addrlen);
DEBUGASSERT(state.valuelen <= state.valuelen_nontrunc);
if (ret >= 0)
{
/* Store length of 'from' address that was available at
* daemon-side.
*/
outaddrlen = state.valuelen_nontrunc;
/* If the MSG_PEEK flag is enabled, it will only peek
* from the buffer, so remark the input as ready.
*/
if (flags & MSG_PEEK)
{
conn->flags |= USRSOCK_EVENT_RECVFROM_AVAIL;
}
}
}
usrsock_teardown_datain(conn);
usrsock_teardown_data_request_callback(&state);
}
while (ret == -EAGAIN);
errout_unlock:
net_unlock();
if (fromlen)
{
*fromlen = outaddrlen;
}
return ret;
}
#endif /* CONFIG_NET && CONFIG_NET_USRSOCK */