Opened 5 years ago

Closed 5 years ago

#1779 closed enhancement (fixed)

[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 (1)

printf_format.patch (742 bytes) - added by ettl.martin 5 years ago.
proposed patch

Download all attachments as: .zip

Change History (3)

Changed 5 years ago by ettl.martin

proposed patch

comment:1 Changed 5 years ago by cehoyos

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

(What problem does your patch fix?)

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

comment:2 Changed 5 years ago by richardpl

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

patch sent to mailing list.

Note: See TracTickets for help on using tickets.