diff options
author | patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3> | 2007-11-23 13:31:28 +0000 |
---|---|---|
committer | patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3> | 2007-11-23 13:31:28 +0000 |
commit | 403a4936c8fc3756003a28328c18c770bc914b6d (patch) | |
tree | 1f09ba2f05bd7c92efe529d4f4bc0a065b4b3298 /nuttx/net | |
parent | 797f58d1f0e2265f7c3c054eb1b8f7bfe4fcecfc (diff) | |
download | px4-nuttx-403a4936c8fc3756003a28328c18c770bc914b6d.tar.gz px4-nuttx-403a4936c8fc3756003a28328c18c770bc914b6d.tar.bz2 px4-nuttx-403a4936c8fc3756003a28328c18c770bc914b6d.zip |
Debug UDP send logic
git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@401 42af7a65-404d-4744-a932-0658087f49c3
Diffstat (limited to 'nuttx/net')
-rw-r--r-- | nuttx/net/sendto.c | 15 | ||||
-rw-r--r-- | nuttx/net/uip/uip-send.c | 8 | ||||
-rw-r--r-- | nuttx/net/uip/uip-udpsend.c | 17 |
3 files changed, 23 insertions, 17 deletions
diff --git a/nuttx/net/sendto.c b/nuttx/net/sendto.c index e6f267a69..ba9bac0af 100644 --- a/nuttx/net/sendto.c +++ b/nuttx/net/sendto.c @@ -95,18 +95,19 @@ void sendto_interrupt(struct uip_driver_s *dev, struct uip_udp_conn *conn, uint8 struct sendto_s *pstate = (struct sendto_s *)conn->private; if (pstate) { - /* Check if the connectin was rejected */ + /* Check if the connection was rejected */ if ((flags & (UIP_CLOSE|UIP_ABORT|UIP_TIMEDOUT)) != 0) { + /* Yes.. then terminate with an error */ + pstate->st_sndlen = -ENOTCONN; } else { - /* Copy the user data into d_appdata and send it */ + /* No.. Copy the user data into d_snddata and send it */ - memcpy(dev->d_appdata, pstate->st_buffer, pstate->st_buflen); - uip_send(dev, dev->d_appdata, pstate->st_buflen); + uip_send(dev, pstate->st_buffer, pstate->st_buflen); pstate->st_sndlen = pstate->st_buflen; } @@ -264,7 +265,7 @@ ssize_t sendto(int sockfd, const void *buf, size_t len, int flags, * are ready. */ - save = irqsave(); + save = irqsave(); memset(&state, 0, sizeof(struct sendto_s)); sem_init(&state.st_sem, 0, 0); state.st_buflen = len; @@ -282,7 +283,7 @@ ssize_t sendto(int sockfd, const void *buf, size_t len, int flags, /* Set up the callback in the connection */ - udp_conn = (struct uip_udp_conn *)psock->s_conn; + udp_conn = (struct uip_udp_conn *)psock->s_conn; udp_conn->private = (void*)&state; udp_conn->event = sendto_interrupt; @@ -290,7 +291,7 @@ ssize_t sendto(int sockfd, const void *buf, size_t len, int flags, uip_udpenable(psock->s_conn); - /* Notify the device driver of the availaibilty of TX data */ + /* Notify the device driver of the availabilty of TX data */ netdev_txnotify(&udp_conn->ripaddr); diff --git a/nuttx/net/uip/uip-send.c b/nuttx/net/uip/uip-send.c index 93096168a..c9899db9e 100644 --- a/nuttx/net/uip/uip-send.c +++ b/nuttx/net/uip/uip-send.c @@ -93,9 +93,13 @@ void uip_send(struct uip_driver_s *dev, const void *buf, int len) { + /* Some sanity checks -- note that the actually available length in the + * buffer is considerably less than CONFIG_NET_BUFSIZE. + */ + if (dev && len > 0 && len < CONFIG_NET_BUFSIZE) { + memcpy(dev->d_snddata, buf, len); dev->d_sndlen = len; - memcpy(dev->d_snddata, buf, len ); - } + } } diff --git a/nuttx/net/uip/uip-udpsend.c b/nuttx/net/uip/uip-udpsend.c index 8f77f78a2..8936c64bd 100644 --- a/nuttx/net/uip/uip-udpsend.c +++ b/nuttx/net/uip/uip-udpsend.c @@ -95,7 +95,7 @@ void uip_udpsend(struct uip_driver_s *dev, struct uip_udp_conn *conn) { - if (dev->d_sndlen == 0) + if (dev->d_sndlen > 0) { /* The total lenth to send is the size of the application data plus * the IP and UDP headers (and, eventually, the ethernet header) @@ -131,8 +131,8 @@ void uip_udpsend(struct uip_driver_s *dev, struct uip_udp_conn *conn) UDPBUF->ipid[1] = g_ipid & 0xff; UDPBUF->ipoffset[0] = 0; UDPBUF->ipoffset[1] = 0; - UDPBUF->ttl = conn->ttl; - UDPBUF->proto = UIP_PROTO_UDP; + UDPBUF->ttl = conn->ttl; + UDPBUF->proto = UIP_PROTO_UDP; /* Calculate IP checksum. */ @@ -146,20 +146,21 @@ void uip_udpsend(struct uip_driver_s *dev, struct uip_udp_conn *conn) /* Initialize the UDP header */ - UDPBUF->srcport = conn->lport; - UDPBUF->destport = conn->rport; - UDPBUF->udplen = HTONS(dev->d_sndlen + UIP_UDPH_LEN); + UDPBUF->srcport = conn->lport; + UDPBUF->destport = conn->rport; + UDPBUF->udplen = HTONS(dev->d_sndlen + UIP_UDPH_LEN); #ifdef CONFIG_NET_UDP_CHECKSUMS /* Calculate UDP checksum. */ - UDPBUF->udpchksum = ~(uip_udpchksum(dev)); + UDPBUF->udpchksum = 0; + UDPBUF->udpchksum = ~(uip_udpchksum(dev)); if (UDPBUF->udpchksum == 0) { UDPBUF->udpchksum = 0xffff; } #else - UDPBUF->udpchksum = 0; + UDPBUF->udpchksum = 0; #endif vdbg("Outgoing UDP packet length: %d (%d)\n", |