]> git.sesse.net Git - ffmpeg/commit
avformat/rtpdec_jpeg: fix low contrast image on low quality setting
authorIco Doornekamp <libav@zevv.nl>
Thu, 24 Mar 2016 13:31:38 +0000 (14:31 +0100)
committerMichael Niedermayer <michael@niedermayer.cc>
Thu, 24 Mar 2016 14:40:10 +0000 (15:40 +0100)
commite3e6a2cff4af9542455d416faec4584d5e823d5d
treeff89b8654649bda3584fad8ab3e3a608c2a529e8
parent72e13600079ea480ed8b8ae1ac44469590024596
avformat/rtpdec_jpeg: fix low contrast image on low quality setting

Original mail and my own followup on ffmpeg-user earlier today:

I have a device sending out a MJPEG/RTP stream on a low quality setting.
Decoding and displaying the video with libavformat results in a washed
out, low contrast, greyish image. Playing the same stream with VLC results
in proper color representation.

Screenshots for comparison:

  http://zevv.nl/div/libav/shot-ffplay.jpg
  http://zevv.nl/div/libav/shot-vlc.jpg

A pcap capture of a few seconds of video and SDP file for playing the
stream are available at

  http://zevv.nl/div/libav/mjpeg.pcap
  http://zevv.nl/div/libav/mjpeg.sdp

I believe the problem might be in the calculation of the quantization
tables in the function create_default_qtables(), the attached patch
solves the issue for me.

The problem is that the argument 'q' is of the type uint8_t. According to the
JPEG standard, if 1 <= q <= 50, the scale factor 'S' should be 5000 / Q.
Because the create_default_qtables() reuses the variable 'q' to store the
result of this calculation, for small values of q < 19, q wil subsequently
overflow and give wrong results in the calculated quantization tables. The
patch below uses a new variable 'S' (same name as in RFC2435) with the proper
range to store the result of the division.

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
libavformat/rtpdec_jpeg.c