Bug 1816 – IP fragmentation should keep packet tags

Bug 1816 - IP fragmentation should keep packet tags


Summary:	IP fragmentation should keep packet tags

Status:	RESOLVED FIXED

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

Importance:	P5 enhancement
Assigned To:	George Riley

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2013-12-20 03:16 UTC by Tommaso Pecorella
Modified:	2014-02-20 16:43 UTC (History)
CC List:	2 users (show)

See Also:

Attachments
Test for IPv4 (5.40 KB, application/octet-stream) 2013-12-20 03:17 UTC, Tommaso Pecorella	Details
Test for IPv6 (5.53 KB, application/octet-stream) 2013-12-20 03:18 UTC, Tommaso Pecorella	Details
IPv[4,6] fragmentation tests now probe for tags as well. (7.54 KB, patch) 2013-12-20 06:34 UTC, Tommaso Pecorella	Details \| Diff
Show Obsolete (2) 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 Tommaso Pecorella 2013-12-20 03:16:23 UTC

In this thread (https://groups.google.com/forum/#!topic/ns-3-users/IWs4QKcHdYg) a user is pointing out that tags in packets are not saved across IP
fragmentation.

This has been patched in changeset 10539:3bb6aa44d1f6.

A test suite is needed.

Comment 1 Tommaso Pecorella 2013-12-20 03:17:59 UTC

Created attachment 1740 [details]
Test for IPv4

Comment 2 Tommaso Pecorella 2013-12-20 03:18:26 UTC

Created attachment 1741 [details]
Test for IPv6

Comment 3 Tommaso Pecorella 2013-12-20 06:34:31 UTC

Created attachment 1742 [details]
IPv[4,6] fragmentation tests now probe for tags as well.

Comment 4 Tom Henderson 2014-01-13 02:08:30 UTC

What is being proposed to merge here?  All three patches?  Why not just the last one (that patches a test suite)?

Comment 5 Tommaso Pecorella 2014-01-13 03:40:24 UTC

Just the last one, of course. Forgot to classify the other 2 as obsolete.

T.

Comment 6 Tommaso Pecorella 2014-02-20 16:43:52 UTC

Pushed the tests in changeset:   10620:30da3895c9b5