serial: Don't append serial header to received packets
In change c67605bc9, we separated the serial header & tailer from the
actual MCTP packet data. However, in the RX path we still included the
header, which now consumes more than the allocated header space,
resulting in an assertion in the RX path.
This change removes the serial-specific header data from the incoming
MCTP packet.
Reported-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
diff --git a/serial.c b/serial.c
index c9f6c90..2fa3abe 100644
--- a/serial.c
+++ b/serial.c
@@ -185,14 +185,8 @@
mctp_prdebug("invalid size %d", c);
serial->rx_state = STATE_WAIT_SYNC_START;
} else {
- uint8_t *p;
-
mctp_serial_start_packet(serial, 0);
pkt = serial->rx_pkt;
- p = mctp_pktbuf_alloc_start(pkt, 3);
- p[0] = MCTP_SERIAL_FRAMING_FLAG;
- p[1] = MCTP_SERIAL_REVISION;
- p[2] = c;
serial->rx_exp_len = c;
serial->rx_state = STATE_DATA;
}