[Poppler-bugs] [Bug 99416] Sign PDF with digital signature
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Fri Jun 16 07:45:19 UTC 2017
https://bugs.freedesktop.org/show_bug.cgi?id=99416
--- Comment #8 from Hans-Ulrich Jüttner <huj at froreich-bioscientia.de> ---
(In reply to oliver.sander from comment #7)
> I tried the patch, but it doesn't apply cleanly. I suppose that's because
> it depends on some incarnation of the patches in
>
> https://bugs.freedesktop.org/show_bug.cgi?id=99271
>
> So getting those in would help with reviewing here.
You are right, this patch depends heavily on the patch #99271.
Unfortunately it started off from that patch as it was in comment 7
before rework after your review on #99271. That results in not applying
cleanly now for this patch. There seems to be hope that we get patch
#99271 in some time soon. Therefore I suggest that I'll revise this
patch then on a clean basis with attention to your comments below.
[snip]
> @@ -535,7 +591,7 @@ GooString* FormWidgetSignature::getCheckedSignature()
> break;
> }
> }
> - if (sigLen > 0 && 2*(sigLen+lenBytes) < len-4)
> + if (sigLen > 0 && 2*(sigLen+lenBytes) <= len-4)
>
>
> This smells like a separate bugfix. Should this go into a separate patch?
>
Yes this should go into patch #99271 as getCheckedSignature() was added there.
[snip]
Thank you for the review!
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/poppler-bugs/attachments/20170616/17a431a9/attachment.html>
More information about the Poppler-bugs
mailing list