Project

General

Profile

Bug #12170

rawspeed commit # 1 broke the decoding of the image

Added by Mike Romanov over 1 year ago. Updated about 1 year ago.

Status:
Fixed
Priority:
Low
Assignee:
-
Category:
-
Target version:
Start date:
04/30/2018
Due date:
% Done:

100%

Estimated time:
Affected Version:
git master branch
System:
all
bitness:
64-bit
hardware architecture:
amd64/x86

Description

rawspeed commit 31c3a9a4ddaff1af40a336ffa11e3ff657b3b028 broke the decoding of the 14-bit lossless and 12-bit lossy image:

[rawspeed] (20180429__DSC0778.NEF) rawspeed::uint32 rawspeed::HuffmanTable::setNCodesPerLength(const rawspeed::Buffer&), line 149: Too big code-values table
[rawspeed] (20180429__DSC0777.NEF) rawspeed::uint32 rawspeed::HuffmanTable::setNCodesPerLength(const rawspeed::Buffer&), line 149: Too big code-values table
[temperature] failed to read camera white balance information from `20180429__DSC0777.NEF'!
[temperature] `NIKON CORPORATION NIKON D7200' color matrix not found for image
[temperature] failed to read camera white balance information from `20180429__DSC0777.NEF'!
[temperature] failed to read camera white balance information from `20180429__DSC0777.NEF'!
allocation failed???
[temperature] `NIKON CORPORATION NIKON D3400' color matrix not found for image
[temperature] failed to read camera white balance information from `20171202__DSC1760.NEF'!

History

#1 Updated by Mike Romanov over 1 year ago

Sorry, 12-bit lossy image is ok (that was ghost in base).

#2 Updated by Roman Lebedev over 1 year ago

  • % Done changed from 0 to 20
  • Status changed from New to Incomplete

Mike Romanov wrote:

rawspeed commit 31c3a9a4ddaff1af40a336ffa11e3ff657b3b028 broke the decoding of the 14-bit lossless and 12-bit lossy image:

[...]

1. Are you sure about that commit specifically?
2. Samples?
3. Why are you using some weird commit of git repo anyway? You should use the one specified by darktable, see https://github.com/darktable-org/darktable#building
git submodule init
git submodule update

#3 Updated by Roman Lebedev over 1 year ago

  • % Done changed from 20 to 50
  • Status changed from Incomplete to In Progress

Oh, hmm, i have a pretty good guess on what's going on.
Thanks for the report :/

#5 Updated by Roman Lebedev over 1 year ago

  • % Done changed from 50 to 100
  • Status changed from In Progress to Fixed

Since this was never in the dt, this was never broken :)

#6 Updated by Mike Romanov over 1 year ago

I update rawspeed separately and was able to prevent this in dt :D

Thank you!

#7 Updated by Roman Lebedev over 1 year ago

Mike Romanov wrote:

I update rawspeed separately and was able to prevent this in dt :D

Thank you!

oss-fuzz would most likely have caught it soon anyway, before it being propagated into dt.

#8 Updated by Roman Lebedev about 1 year ago

  • Target version set to 2.6.0

Also available in: Atom PDF

Go to top