Skip to content

Commit ebcb099

Browse files
[core] Coverity 1325269 Dereference before null check
1 parent b6ccc27 commit ebcb099

File tree

1 file changed

+6
-13
lines changed

1 file changed

+6
-13
lines changed

src/switch_jitterbuffer.c

+6-13
Original file line numberDiff line numberDiff line change
@@ -1719,20 +1719,13 @@ SWITCH_DECLARE(switch_status_t) switch_jb_get_packet(switch_jb_t *jb, switch_rtp
17191719
}
17201720
}
17211721

1722-
if (node) {
1723-
status = SWITCH_STATUS_SUCCESS;
1722+
*packet = node->packet;
1723+
*len = node->len;
1724+
jb->last_len = *len;
1725+
packet->header.version = 2;
1726+
hide_node(node, SWITCH_TRUE);
17241727

1725-
*packet = node->packet;
1726-
*len = node->len;
1727-
jb->last_len = *len;
1728-
packet->header.version = 2;
1729-
hide_node(node, SWITCH_TRUE);
1730-
1731-
jb_debug(jb, 2, "GET packet ts:%u seq:%u %s\n", ntohl(packet->header.ts), ntohs(packet->header.seq), packet->header.m ? " <MARK>" : "");
1732-
1733-
} else {
1734-
status = SWITCH_STATUS_MORE_DATA;
1735-
}
1728+
jb_debug(jb, 2, "GET packet ts:%u seq:%u %s\n", ntohl(packet->header.ts), ntohs(packet->header.seq), packet->header.m ? " <MARK>" : "");
17361729

17371730
end:
17381731

0 commit comments

Comments
 (0)