pfring: update PfringThreadVars_ for gcc 4.x

[jmtaylor90]

Signed-off-by: jason taylor jtfas90@gmail.com

Make sure these boxes are signed before submitting your Pull Request -- thank you.

• I have read the contributing guide lines at https://redmine.openinfosecfoundation.org/projects/suricata/wiki/Contributing
• I have signed the Open Information Security Foundation contribution agreement at https://suricata-ids.org/about/contribution-agreement/
• I have updated the user guide (in doc/userguide/) to reflect the changes made (if applicable)

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/2899

Describe changes:

• In the 4.4.7 version of gcc on el6 there is stricter checking via -pedantic than newer versions. This change removes the duplicate typedef keyword and relies on the definition in the header file.

PRScript output (if applicable):

[victorjulien]

As PfringThreadVars are only used in source-pfring.c I would like to suggest removing the typedef from source-pfring.h instead.

[jmtaylor90]

As PfringThreadVars are only used in source-pfring.c I would like to suggest removing the typedef from source-pfring.h instead.

What about the PfringPacketVars_ struct in src/source-pfring.h?

typedef struct PfringPacketVars_
{
    PfringThreadVars *ptv;
    uint32_t flow_id;
} PfringPacketVars;

[victorjulien]

Ah I missed that, sorry.

[victorjulien]

Merged through #3783, thanks Jason!