Improve BFCP detection

[0xA50C1A1]

Please sign (check) the below before submitting the Pull Request:

• I have signed the ntop Contributor License Agreement at https://github.com/ntop/legal/blob/main/individual-contributor-licence-agreement.md
• I have read the contributing guidelines at https://github.com/ntop/nDPI/blob/dev/CONTRIBUTING.md
• I have updated the documentation (in doc/) to reflect the changes made (if applicable)

Link to the related issue: #2745

Describe changes:

• Added a check for the Payload Length field to ensure it’s correct.
• Fixed the validation for the primitive field.
• Added a valid PCAP sample for BFCP over TCP, sourced from a libbfcp example.

Unfortunately, I couldn't merge the PCAP examples as ndpiReader throws an error when trying to read the final file. I’ve included two files: bfcp_tcp.pcapng and bfcp_udp.pcapng.

[IvanNardi]

@0xA50C1A1, I am facing some issue locally with this PR... could you try to rebase, please?

[0xA50C1A1]

@0xA50C1A1, I am facing some issue locally with this PR... could you try to rebase, please?

Try it now, hope it works.

[IvanNardi]

@0xA50C1A1, I merged the two traces. Could you take a look if everything is still fine, please?

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[0xA50C1A1]

@0xA50C1A1, I merged the two traces. Could you take a look if everything is still fine, please?

Thanks, everything looks ok.

[IvanNardi]

Thank you