On Wed, 7 Sep 2005 [EMAIL PROTECTED] wrote:

 Had a coredump where ACLIP::data was NULL.  This patch avoids the
 NULL pointer dereference, but there might be another bug elsewhere that
 allows data to be NULL.

ACLs are allowed to be empty these days. This was changed during the code sprint to make configuration consistent when there is empty acls.

From what I can tell the change you made should not make any difference.
SplayNode<V>::destroy already checks for NULL and should catch this, and it is not a virtual method. But on the other hand the change does not hurt either, just duplicates some code.

Do you have any additional information on the core dump?

Regards
Henrik

Reply via email to