ref: e1c687a1274a8d3073eea8232346acc15ff35a87
parent: 8a75eb978c410897fa934af785238422aae8284f
author: Ralph Giles <giles@mozilla.com>
date: Wed Aug 22 13:21:57 EDT 2012
Print each header on one line. This makes the capture output a little less verbose so it's easier to read. Also corrects the rtp payload length (the udp header's size includes the udp header bytes) and removes a redundant check for payload type 109.
--- a/src/opusrtp.c
+++ b/src/opusrtp.c
@@ -212,13 +212,13 @@
memcpy(eth.dst, packet + 6, 6);
eth.type = packet[12] << 8 | packet[13];
- fprintf(stderr, " src mac %02x:%02x:%02x:%02x:%02x:%02x\n",
+ fprintf(stderr, " eth 0x%04x", eth.type);
+ fprintf(stderr, " %02x:%02x:%02x:%02x:%02x:%02x ->",
eth.src[0], eth.src[1], eth.src[2],
eth.src[3], eth.src[4], eth.src[5]);
- fprintf(stderr, " dst mac %02x:%02x:%02x:%02x:%02x:%02x\n",
+ fprintf(stderr, " %02x:%02x:%02x:%02x:%02x:%02x\n",
eth.dst[0], eth.dst[1], eth.dst[2],
eth.dst[3], eth.dst[4], eth.dst[5]);
- fprintf(stderr, " eth type 0x%04x\n", eth.type);
/* ipv4 header */
if (header->caplen < 14 + 20) {
@@ -229,17 +229,14 @@
ip.header_size = 4 * (packet[14+0] & 0x0f);
ip.protocol = packet[14 + 9];
memcpy(ip.src, packet + 14 + 12, 4);
- memcpy(ip.dst, packet + 12 + 16, 4);
+ memcpy(ip.dst, packet + 14 + 16, 4);
-#if 0
- fprintf(stderr, " IP version %d\n", ip.version);
- fprintf(stderr, " header length %d\n", ip.header_size);
- fprintf(stderr, " src addr %d.%d.%d.%d\n",
+ fprintf(stderr, " ipv%d protocol %d", ip.version, ip.protocol);
+ fprintf(stderr, " %d.%d.%d.%d ->",
ip.src[0], ip.src[1], ip.src[2], ip.src[3]);
- fprintf(stderr, " dst addr %d.%d.%d.%d\n",
+ fprintf(stderr, " %d.%d.%d.%d",
ip.dst[0], ip.dst[1], ip.dst[2], ip.dst[3]);
- fprintf(stderr, " protocol %d\n", ip.protocol);
-#endif
+ fprintf(stderr, " header %d bytes\n", ip.header_size);
if (header->caplen < 14 + ip.header_size) {
fprintf(stderr, "Packet too short for ipv4 with options\n");
return;
@@ -257,13 +254,8 @@
packet[14+ip.header_size + 5];
udp.checksum = packet[14+ip.header_size + 6] << 8 |
packet[14+ip.header_size + 7];
-#if 0
- fprintf(stderr, " src port %d\n", udp.src);
- fprintf(stderr, " dst port %d\n", udp.dst);
- fprintf(stderr, " udp length %d\n", udp.size);
- fprintf(stderr, " checksum %d\n", udp.checksum);
-#endif
-
+ fprintf(stderr, " udp %d bytes %d -> %d crc 0x%04x\n",
+ udp.size, udp.src, udp.dst, udp.checksum);
if (header->caplen < 14 + ip.header_size + 8 + 12) {
fprintf(stderr, "Packet too short for rtp\n");
return;
@@ -273,7 +265,7 @@
rtp.ext = (packet[14+ip.header_size+8] >> 4) & 1;
rtp.cc = packet[14+ip.header_size+8] & 7;
rtp.header_size = 12 + 4 * rtp.cc;
- rtp.payload_size = udp.size - rtp.header_size;
+ rtp.payload_size = udp.size - 8 - rtp.header_size;
rtp.mark = (packet[14+ip.header_size+8 + 1] >> 7) & 1;
rtp.type = (packet[14+ip.header_size+8 + 1]) & 127;
@@ -288,17 +280,12 @@
packet[14+ip.header_size+8 + 10] << 8 |
packet[14+ip.header_size+8 + 11];
- fprintf(stderr, " rtp version %d\n", rtp.version);
- fprintf(stderr, " payload type %d\n", rtp.type);
- fprintf(stderr, " SSRC 0x%08x\n", rtp.ssrc);
- fprintf(stderr, " sequence no %d\n", rtp.seq);
- fprintf(stderr, " timestamp %d\n", rtp.time);
-#if 0
- fprintf(stderr, " padding %d\n", rtp.pad);
- fprintf(stderr, " extension %d\n", rtp.ext);
- fprintf(stderr, " CSRC count %d\n", rtp.cc);
- fprintf(stderr, " marker %d\n", rtp.mark);
-#endif
+ fprintf(stderr, " rtp 0x%08x %d %d %d",
+ rtp.ssrc, rtp.type, rtp.seq, rtp.time);
+ fprintf(stderr, " v%d %s%s%s CC %d", rtp.version,
+ rtp.pad ? "P":".", rtp.ext ? "X":".",
+ rtp.mark ? "M":".", rtp.cc);
+ fprintf(stderr, " %5d bytes\n", rtp.payload_size);
if (header->caplen < 14 + ip.header_size + 8 + rtp.header_size) {
fprintf(stderr, "skipping short packet\n");
@@ -319,17 +306,16 @@
/* write the payload to our opus file */
int size = header->caplen - 14 - ip.header_size - 8 - rtp.header_size;
unsigned char *data = packet + 14+ip.header_size+8 + rtp.header_size;
- fprintf(stderr, " payload data %d bytes\n",
- header->len - 14 - ip.header_size - 8 - rtp.header_size);
- if ( (packet[14+ip.header_size + 8 + 1] & 127) != 109) {
- fprintf(stderr, "skipping non-opus packet\n");
- return;
- }
ogg_packet *op = op_from_pkt(data, size);
op->granulepos = 960*rtp.seq;
ogg_stream_packetin(params->stream, op);
free(op);
ogg_write(params);
+
+ if (size < rtp.payload_size) {
+ fprintf(stderr, "!! truncated %d uncaptured bytes\n",
+ rtp.payload_size - size);
+ }
}
int main(int argc, char *argv[])