We *MUST NOT* free the data pointer explicitly. Anything that allocat…

…es and expects it to be freed MUST allocate from the context of the packet. This caused abort() in radsniff
FreeRADIUS · Jul 23, 2013 · f1be0ba · f1be0ba
1 parent 16fbccf
commit f1be0ba
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/src/lib/radius.c b/src/lib/radius.c
@@ -4581,8 +4581,6 @@ void rad_free(RADIUS_PACKET **radius_packet_ptr)
 	if (!radius_packet_ptr || !*radius_packet_ptr) return;
 	radius_packet = *radius_packet_ptr;
 
-	talloc_free(radius_packet->data); /* not really necessary... */
-
 	pairfree(&radius_packet->vps);
 
 	talloc_free(radius_packet);