[asterisk-bugs] [JIRA] (ASTERISK-28921) Wrong return value check for fwrite when writing to pcap file

[nappsoft (JIRA)]

nappsoft created ASTERISK-28921:
-----------------------------------

             Summary: Wrong return value check for fwrite when writing to pcap file
                 Key: ASTERISK-28921
                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-28921
             Project: Asterisk
          Issue Type: Bug
      Security Level: None
          Components: Resources/res_pjsip_logger
    Affects Versions: 16.11.0
            Reporter: nappsoft
            Severity: Minor


When using the new feature that logs to pcap files one will get 5 lines with warnings for each packet written.

This is because of the fact that the wrong return value is being checked: fwrite does not return the number of bytes written, but the number of elements written. However asterisk is comparing the return value (which will always be 1) with the size of the element, resulting in the following output:

[May 29 07:22:23] WARNING[7274]: res_pjsip_logger.c:250 pjsip_logger_write_to_pcap: Writing PCAP header failed: Invalid argument
[May 29 07:22:23] WARNING[7274]: res_pjsip_logger.c:253 pjsip_logger_write_to_pcap: Writing ethernet header to pcap failed: Invalid argument
[May 29 07:22:23] WARNING[7274]: res_pjsip_logger.c:256 pjsip_logger_write_to_pcap: Writing IP header to pcap failed: Invalid argument
[May 29 07:22:23] WARNING[7274]: res_pjsip_logger.c:259 pjsip_logger_write_to_pcap: Writing UDP header to pcap failed: Invalid argument
[May 29 07:22:23] WARNING[7274]: res_pjsip_logger.c:262 pjsip_logger_write_to_pcap: Writing UDP payload to pcap failed: Invalid argument

The attached patch fixes this.



--
This message was sent by Atlassian JIRA
(v6.2#6252)