fix: more ICC protections against invalid tag sizes #4565
Conversation
Signed-off-by: Larry Gritz <lg@larrygritz.com>
| oiio:ColorSpace: "sRGB" | ||
| corrupt-icc-4551.jpg | ||
| DCT coefficient (lossy) or spatial difference (lossless) out of range | ||
| Reading src/corrupt-icc-4552.jpg |
There was a problem hiding this comment.
Quick question: since the file we're reading here is supposed to be corrupted, shouldn't we be failing when reading it ?
There was a problem hiding this comment.
Well, there's a balance -- if there's something wrong with one tag in the ICC profile, which is just one (non-essential to many apps) piece of metadata in the file, do we press on and try to read everything else that we can for the file? Or upon the first problem, no matter how minor, do we conservatively abort because it might be a sign that the whole file is suspicious or maybe even malicious? We usually err on the side of trying to read as much as we can, unless the whole file is unrecoverably broken.
This is the point of #4560: adding an attribute that, in the future, will give an app using OIIO more control over tradeoff I just described.
There was a problem hiding this comment.
That makes sense !
LGTM then :)
…oundation#4565) Signed-off-by: Larry Gritz <lg@larrygritz.com>
|
Noting for the record that this turns out to also have fixed #4450. |
…oundation#4565) Signed-off-by: Larry Gritz <lg@larrygritz.com> Signed-off-by: Scott Wilson <scott@propersquid.com>
2 participants
No description provided.