Bug 2016

Summary:	Radvd do not consider the SendAdvert option and don't reply to RS
Product:	ns-3	Reporter:	Tommaso Pecorella <tommaso.pecorella>
Component:	applications	Assignee:	George Riley <riley>
Status:	RESOLVED FIXED
Severity:	normal	CC:	ns-bugs, tomh
Priority:	P5
Version:	ns-3-dev
Hardware:	All
OS:	All
Attachments:	patch patch new patch

Description Tommaso Pecorella 2014-11-09 17:00:49 UTC

Radvd simply ignores the fact that SendAdvert is turned off.
The attached patch fixes this misbehaviour.

Comment 1 Tommaso Pecorella 2014-11-09 17:02:00 UTC

Created attachment 1916 [details]
patch

Comment 2 Tommaso Pecorella 2014-11-14 04:43:58 UTC

Created attachment 1917 [details]
patch

It turns out that raided had more bugs than expected.
1) RS are not correctly received and parsed
2) when an RS parsed, the reply is sent on all the interfaces, causing a NS flood on the "wrong" interfaces

This patch fixes all the issues.
Moreover, it updates the raided example to show how to disable automatic RA on one of the two interfaces.

Comment 3 Tommaso Pecorella 2014-11-16 12:16:07 UTC

Created attachment 1921 [details]
new patch

Better fix, following more strictly what's suggested in RFC 4861

Comment 4 Tom Henderson 2014-11-17 19:59:31 UTC

please merge when ready (p.s. minor nit:  you still have magic numbers instead of constants in your constructor initializers)

Comment 5 Tommaso Pecorella 2014-11-18 18:17:54 UTC

changeset:   11062:91a7bee99048

The magic numbers are the ones from RFC 4861, and they should be converted into Attributes (maybe). I'll do it in the next patch if there is interest in tweaking RADVD params.