Bug 2332 – Assert failed when using aggregation and two pairs of AP-STA

Bug 2332 - Assert failed when using aggregation and two pairs of AP-STA


Summary:	Assert failed when using aggregation and two pairs of AP-STA

Status:	RESOLVED FIXED

Product:	ns-3
Classification:	Unclassified
Component:	wifi
Version:	ns-3-dev
Hardware:	All All

Importance:	P5 major
Assigned To:	sebastien.deronne

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2016-03-09 11:47 UTC by Matías Richart
Modified:	2016-03-09 16:03 UTC (History)
CC List:	1 user (show)

See Also:

Attachments
Patch to reproduce the bug (1.40 KB, patch) 2016-03-09 11:47 UTC, Matías Richart	Details \| Diff
patch to fix (880 bytes, patch) 2016-03-09 15:42 UTC, sebastien.deronne	Details \| Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Matías Richart 2016-03-09 11:47:42 UTC

Created attachment 2335 [details]
Patch to reproduce the bug

The assert that rises:
assert failed. cond="m_buffer.GetSize () >= start + length", file=../src/network/model/packet.cc, line=234

It happens in a scenario with two pairs of AP-STA where each AP is sending data to the associated STA.
I attach a patch of wifi-aggregation example to reproduce the issue.

The problem appears after commit 11945:96b721d35158 which fixes bug 2313.

Comment 1 sebastien.deronne 2016-03-09 15:42:47 UTC

Created attachment 2337 [details]
patch to fix

Add missing check when fixing bug 2313 (commit 96b721d35158)

Comment 2 sebastien.deronne 2016-03-09 15:43:36 UTC

Obvious fix, I will deliver this one in my bunch of patch for the RC.

Comment 3 sebastien.deronne 2016-03-09 16:03:11 UTC

fixed in changeset 12003:3b4340478fc1