|
Bugzilla – Full Text Bug Listing |
| Summary: | NdiscCache use slightly bugged... | ||
|---|---|---|---|
| Product: | ns-3 | Reporter: | Tommaso Pecorella <tommaso.pecorella> |
| Component: | ipv6 | Assignee: | Tommaso Pecorella <tommaso.pecorella> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | ns-bugs |
| Priority: | P5 | ||
| Version: | ns-3-dev | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | |||
| Bug Blocks: | 1646 | ||
| Attachments: |
patch
This is the right patch. Sorry Another try... |
||
Created attachment 1568 [details]
This is the right patch. Sorry
Created attachment 1569 [details]
Another try...
Bugzilla seems to cut the patch file. Or it's my Mac ?
By the way, the rationale of this patch is this: there are some cases where the NdicCache is not created (i,e., for the loopback). Hence it's mandatory to check the cache existence, otherwise a dumb user (me) could try to look for it on the wrong interfaces. This greatly simplifies the code and makes it easier to check for an NDISC entry on an unknown interface. Fixed in changeset 9760 - 858d18e11e11 |
Created attachment 1566 [details] patch Nothing big, however... 1) It is created also for No-Arp interfaces, which is a resource waste, 2) Icmpv6L4Protocol isn't checking for its existence when performing a lookup, and 3) Icmpv6L4Protocol::Lookup doesn't return the Mac Address in some cases. All the 3 points are fixed in the patch.