riot-ci/RIOT
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

shell: set NULL pointer for data pakets with zero-length payload

Browse Source
This commit is contained in:
Aaron Sowry 2016-06-09 10:01:17 +12:00
parent 29f0eea2ef
commit 1ae7bc23d8
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
sys/shell/commands/sc_netif.c
View File

@ -1041,7 +1041,7 @@ int _netif_send(int argc, char **argv)
{ {
kernel_pid_t dev; kernel_pid_t dev;
uint8_t addr[MAX_ADDR_LEN]; uint8_t addr[MAX_ADDR_LEN];
size_t addr_len; size_t addr_len, data_len;
gnrc_pktsnip_t *pkt, *hdr; gnrc_pktsnip_t *pkt, *hdr;
gnrc_netif_hdr_t *nethdr; gnrc_netif_hdr_t *nethdr;
uint8_t flags = 0x00; uint8_t flags = 0x00;
@ -1073,10 +1073,16 @@ int _netif_send(int argc, char **argv)
} }
/* put packet together */ /* put packet together */
pkt = gnrc_pktbuf_add(NULL, argv[3], strlen(argv[3]), GNRC_NETTYPE_UNDEF); data_len = strlen(argv[3]);
if (pkt == NULL) { if (data_len == 0) {
puts("error: packet buffer full"); pkt = NULL;
return 1; }
else {
pkt = gnrc_pktbuf_add(NULL, argv[3], data_len, GNRC_NETTYPE_UNDEF);
if (pkt == NULL) {
puts("error: packet buffer full");
return 1;
}
} }
hdr = gnrc_netif_hdr_build(NULL, 0, addr, addr_len); hdr = gnrc_netif_hdr_build(NULL, 0, addr, addr_len);
if (hdr == NULL) { if (hdr == NULL) {