Ticket #1779 (closed enhancement: fixed)

Opened 8 months ago

Last modified 7 months ago

[patch] fixed wrong printf format identifier in ffmpeg/ffserver.c

Reported by: ettl.martin Owned by:
Priority: normal Component: FFserver
Version: git-master Keywords:
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

During a code review, i found a wrong printf format identifier in file : ffmpeg/ffserver.c. In this case %d has been used to print an unsigned integer. The attached patch uses %u, which is the correct identifier.

Please refer the attache patch.

Best regards and many thanks

Martin Ettl

Reference:
 http://www.cplusplus.com/reference/clibrary/cstdio/printf/

Attachments

printf_format.patch Download (742 bytes) - added by ettl.martin 8 months ago.
proposed patch

Change History

Changed 8 months ago by ettl.martin

proposed patch

comment:1 Changed 8 months ago by cehoyos

  • Keywords printf, patch removed
  • Component changed from FFmpeg to FFserver

(What problem does your patch fix?)

Please send patches to ffmpeg-devel, they generally receive more attention there.

comment:2 Changed 7 months ago by richardpl

  • Status changed from new to closed
  • Resolution set to fixed

patch sent to mailing list.

Note: See TracTickets for help on using tickets.