Commit 501aa061 authored by Roel Kluin's avatar Roel Kluin Committed by David S. Miller
Browse files

3c505: do not set pcb->data.raw beyond its size 



Ensure that we do not set pcb->data.raw beyond its size, print an error message
and return false if we attempt to. A timout message was printed one too early.

Signed-off-by: default avatarRoel Kluin <roel.kluin@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent f82da723

drivers/net/3c505.c

+16 −10
Original line number Diff line number Diff line
@@ -493,21 +493,27 @@ static bool receive_pcb(struct net_device *dev, pcb_struct * pcb) 
	}

	/* read the data */

	spin_lock_irqsave(&adapter->lock, flags);

	i = 0;

	do {

		j = 0;

		while (((stat = get_status(dev->base_addr)) & ACRF) == 0 && j++ < 20000);

		pcb->data.raw[i++] = inb_command(dev->base_addr);

		if (i > MAX_PCB_DATA)

			INVALID_PCB_MSG(i);

	} while ((stat & ASF_PCB_MASK) != ASF_PCB_END && j < 20000);

	for (i = 0; i < MAX_PCB_DATA; i++) {

		for (j = 0; j < 20000; j++) {

			stat = get_status(dev->base_addr);

			if (stat & ACRF)

				break;

		}

		pcb->data.raw[i] = inb_command(dev->base_addr);

		if ((stat & ASF_PCB_MASK) == ASF_PCB_END || j >= 20000)

			break;

	}

	spin_unlock_irqrestore(&adapter->lock, flags);

	if (i >= MAX_PCB_DATA) {

		INVALID_PCB_MSG(i);

		return false;

	}

	if (j >= 20000) {

		TIMEOUT_MSG(__LINE__);

		return false;

	}

	/* woops, the last "data" byte was really the length! */

	total_length = pcb->data.raw[--i];

	/* the last "data" byte was really the length! */

	total_length = pcb->data.raw[i];



	/* safety check total length vs data length */

	if (total_length != (pcb->length + 2)) {

CRA Git | Maintained and supported by SUSTech CRA and CCSE