Opened 12 years ago

Closed 12 years ago

#1779 closed enhancement (fixed)

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

Reported by: Martin Ettl 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 Martin Ettl 12 years ago.
proposed patch

Download all attachments as: .zip

Change History (3)

by Martin Ettl, 12 years ago

Attachment: printf_format.patch added

proposed patch

comment:1 by Carl Eugen Hoyos, 12 years ago

Component: FFmpegFFserver
Keywords: printf patch removed

(What problem does your patch fix?)

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

comment:2 by Elon Musk, 12 years ago

Resolution: fixed
Status: newclosed

patch sent to mailing list.

Note: See TracTickets for help on using tickets.