Commit 332bfea2 authored Dec 23, 2022 by Sarannya S Committed by Gerrit - the friendly Code Review server Jan 04, 2023

net: qrtr: fifo: Add bounds check on tx path

Add bounds check on values read from shared memory in the tx path. In
cases where the VM is misbehaving, the qrtr transport should exit and
print a warning when bogus values may cause out of bounds to be read.

Change-Id: I7ebef28ed8eba4c4da0b32d5114365bbe6bea390
Signed-off-by: Sarannya S <quic_sarannya@quicinc.com>

parent 99fdb93c

net/qrtr/fifo.c

+5 −0

Original line number	Diff line number	Diff line
		@@ -117,6 +117,9 @@ static size_t fifo_tx_avail(struct fifo_pipe *pipe)
		else
		avail = tail - head;

		if (WARN_ON_ONCE(avail > pipe->length))
		avail = 0;

		return avail;
		}

		@@ -127,6 +130,8 @@ static void fifo_tx_write(struct fifo_pipe *pipe,
		u32 head;

		head = le32_to_cpu(*pipe->head);
		if (WARN_ON_ONCE(head > pipe->length))
		return;

		len = min_t(size_t, count, pipe->length - head);
		if (len)