Ship It!

usr/src/cmd/cmd-inet/usr.lib/in.ndpd/main.c (Diff revision 1)

Show all issues

How did you determine that this should always be done. I'm having trouble determining if we're only supposed to be deleting all the prefixes regardless of if that was supposed to be bracketed on the PR_AUTO flag. It may be worth reaching out to Seb about that.

1. 

   Toomas Soome May 16, 2017, 8:59 a.m.

   Since the code is tearing down the interface bits it did sort of made sense, but indeed, I may have been wrong here. Anyhow, I did ask Seb if he can have time to look on this, meanwhile I will also try to dig in more and see if I can get better understanding about this code.

2. 

   Sebastien Roy May 16, 2017, 5:07 p.m.

   The change is correct. The code should be doing (pardon the verbose pseudo-code):
   if physical interface pi removed {
           for each prefix p on pi {
                   if prefix is used for address autoconfiguration {
                           delete addresses created from that prefix
                   }
                   delete the prefix
           }
           cleanup remaining pi state
   }
   

3. 

   Toomas Soome May 16, 2017, 9:12 p.m.

   So as this is the fix afterall, I'll drop this issue.

Ship It!

Ship It!