Project

General

Profile

Bug #12170

rawspeed commit # 1 broke the decoding of the image

Added by Mike Romanov 9 months ago. Updated 5 months ago.

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

100%

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 9 months ago

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

#2 Updated by Roman Lebedev 9 months ago

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

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 9 months 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 9 months ago

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

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

#6 Updated by Mike Romanov 9 months ago

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

Thank you!

#7 Updated by Roman Lebedev 9 months 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 5 months ago

  • Target version set to 2.6.0

Also available in: Atom PDF