Bugzilla – Full Text Bug Listing |
Summary: | Don't process self-directed packets | ||
---|---|---|---|
Product: | ns-3 | Reporter: | John Abraham <john.abraham.in> |
Component: | nix-vector | Assignee: | John Abraham <john.abraham.in> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | jpelkey, ns-bugs, riley, tomh, zhu007 |
Priority: | P5 | ||
Version: | pre-release | ||
Hardware: | All | ||
OS: | All |
Description
John Abraham
2011-12-06 10:34:49 EST
Yes, I believe this is because you are sending packets to yourself and nix-vector does not currently handle this correctly. Personally, I don't see any advantage in Nix-vector-routing having to deal with this special case (its not the primary motivation for use nix-vector) and would prefer static routing or other protocols in the list to handle it, so the following may prevent the error. ns-3-dev john$ hg diff diff -r 9b0902620ff7 src/nix-vector-routing/model/ipv4-nix-vector-routing.cc --- a/src/nix-vector-routing/model/ipv4-nix-vector-routing.cc Tue Dec 06 08:42:44 2011 -0500 +++ b/src/nix-vector-routing/model/ipv4-nix-vector-routing.cc Tue Dec 06 10:27:59 2011 -0500 @@ -140,8 +140,9 @@ // build the nix vector a little differently if (source == destNode) { - BuildNixVectorLocal (nixVector); - return nixVector; + //BuildNixVectorLocal (nixVector); + //return nixVector; + return 0; } else { will commit if no other comments I found out this bug today. @tom , did we commit a bug fix for this yet? your previous comment seems to suggest so (In reply to comment #4) > @tom , did we commit a bug fix for this yet? your previous comment seems to > suggest so Forgot to do this earlier, just pushed in changeset c2649ef48eaa |