Commit 76e8d7b0 authored Oct 30, 2011 by Simon Wunderlich Committed by Sven Eckelmann Nov 20, 2011

batman-adv: Fix range check for expected packets



The check for new packets in the future used a wrong binary operator,
which makes the check expression always true and accepting too many
packets.

Reported-by: Thomas Jarosch <thomas.jarosch@intra2net.com>
Signed-off-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
Signed-off-by: Sven Eckelmann <sven@narfation.org>

parent 8b7342d6

net/batman-adv/bitarray.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -155,7 +155,7 @@ int bit_get_packet(void priv, unsigned long seq_bits,
		/* sequence number is much newer, probably missed a lot of packets */

		if ((seq_num_diff >= TQ_LOCAL_WINDOW_SIZE)
		\|\| (seq_num_diff < EXPECTED_SEQNO_RANGE)) {
		&& (seq_num_diff < EXPECTED_SEQNO_RANGE)) {
		bat_dbg(DBG_BATMAN, bat_priv,
		"We missed a lot of packets (%i) !\n",
		seq_num_diff - 1);