Hi,

On 24/10/2026 00:43, Emmanuel Arias wrote:
I'm sending new debdiff, with your suggestions. Also I pushed to salsa.

Thanks for your review!

On 2026-10-15 11:42, Sylvain Beucler wrote:
On 15/10/2026 04:25, Emmanuel Arias wrote:
On 2026-10-08 14:30, Sylvain Beucler wrote:
On 06/10/2026 21:43, Emmanuel Arias wrote:
I've worked in tiff for bookworm. Please, could you review the patches?

The patches are here:
     https://salsa.debian.org/lts-team/packages/tiff/-/tree/debian/bookworm


Please drop the Bug-Debian-Security and Bug-Freexian-Security which only
repeat default tracker links and aren't informative (in particular the
freexian one while this isn't ELTS).

Ready, I left those that patch name is not CVE-*.patch I backported
those from trixie and I leave as they are.

Less clear/consistent for CVE-2026-12912/pixarlog, but fair enough.


For Salsa-CI, maybe fix debian/watch or add an allow_failure for
'test-uscan', which makes the CI FAIL, which is confusing.

Ready.

The CI is currently not working, related to https://salsa.debian.org/lts-team/pipeline/-/merge_requests/40

I'd like to see the CI running correctly if possible, as the last ones seem to have trouble / flakyness:
https://salsa.debian.org/lts-team/packages/tiff/-/pipelines


CVE-2026-36849 is marked <ignored> in trixie, maybe check with the security
team and/or maintainer if it's a good idea to fix it.

Contacted, thanks!

(So, who did you contact and how?)

I sent an email to the maintainer first, to know the opinion but I
didn't receive any reply yet.

It would be worth knowing why secteam marked it as <ignored>.

Incidentally the CVE is still marked "RESERVED".

If you got an answer that would be enlightening, especially with the following:


I see that the CVE-2026-36849 patch adds a symbol, maybe that's why it was
marked <ignored>, as it's always a delicate change especially after the
bookworm freeze. It also changes a structure. And it's a large patch. Did
you perform specific testing?

Your symbol update seems to be wrong, you mark version 4.5.0 while this is
something you're adding after the bookworm release, in 4.5.0-6+deb12u5 and
not before.

I modify a little bit here to make the function private.

The patch doesn't seem to break the ABI now.

Do you believe we should still backport this DoS fix, including in the 3 ELTS releases?


What kind of testing did you do?
We have some notes from the maintainer at
https://lts-team.pages.debian.net/wiki/TestSuites/tiff.html , did you check
them?
There are many reverse dependencies for this package so thorough testing is
needed.

I uploaded it to debusine [0]. But I trying to reproduce the Test
Suites you mentioned.

[0] https://debusine.debian.net/debian/developers/work-request/1426324/

The tiff package doesn't have autopkgtest and is marked as skipped, so
autopkgtest/debusine won't be enough here.

libgd2, an important package, has all its tests failing here, but I can't
find the reference test results for comparison. Did you investigate what's
happening?

That's pending for me

What's the status?


Try to reproduce individual issues when there's a reproducer, in particular
for CVE-2026-52490 where you hand-picked the patch chunks; test if our
version fails without the patch and doesn't fail anymore with the patch.

I couldn't do that yet, I test it with the bank you mentioned (some of
those images not all)
What's your progress with the reproducers?


Cheers!
Sylvain

Reply via email to