Fixed on branch packet-release-3, which has a new pull request. I closed the pull request for packet-release-2.

From: Victor Julien [mailto:notifications@github.com]
Sent: Tuesday, July 09, 2013 9:47 AM
To: inliniac/suricata
Cc: Kenneth Steele
Subject: Re: [suricata] Replace ReleaseData function pointer on Packet Structure with ReleasePac... (#424)


In src/decode.c:

> @@ -100,6 +113,20 @@ Packet *PacketGetFromAlloc(void)

>  }

>

>  /**

> + * \brief Return a malloced packet.

> + *

> + * \retval p packet, NULL on error

> + */

> +void PacketFreeOrRelease(Packet *p)

> +{

> +  if (p->flags & PKT_ALLOC)

> +    PacketFree(p);

indent is off by 2 space. Need 4, have 2.

—
Reply to this email directly or view it on GitHub<https://github.com/inliniac/suricata/pull/424/files#r5089870>.