|Summary:||[PATCH] editors/openoffice-4: fix nss-related problems with password-protected documents|
|Product:||Ports & Packages||Reporter:||Arrigo Marchiori <ardovm>|
|Component:||Individual Port(s)||Assignee:||Don Lewis <truckman>|
|Severity:||Affects Many People||CC:||damjan.jov, truckman|
Description Arrigo Marchiori 2016-01-14 12:20:11 UTC
Upstream issue 125431 (https://bz.apache.org/ooo/show_bug.cgi?id=125431) reports that password-protected documents may not be opened and saved due to problems with libnss. I was personally encountering the same issue. This bug report is to propose integration of the upstream patch into the FreeBSD port. The patch seems to solve the problem on my system, allowing me to open and save password-protected files again. $ uname -a FreeBSD myhost 9.3-STABLE FreeBSD 9.3-STABLE #140 r293654M: Mon Jan 11 09:55:00 CET 2016 root@myhost:/usr/obj/usr/src/sys/GENERIC i386 $ svn info /usr/ports | grep Revision Revision: 405768
Comment 1 Arrigo Marchiori 2016-01-14 12:27:11 UTC
Created attachment 165563 [details] Upstream patch (vanilla, to be copied into the files/ directory) This file is the ``vanilla'' patch proposed by upstream: https://bz.apache.org/ooo/attachment.cgi?id=84969&action=diff It works if copied into the files/ directory of the editors/openoffice-4 port. I am not sure if it is a good idea to split it into many files, each one patching a single source file, as requested in Porter's Handbook chapter 4.4. In fact, it seems that the files/ directory already contains ``bulk'' patches that affect many files. I am also not sure if it is advisable to apply this fix selectively, i.e. leave to the user the choice between OpenSSL and libnss. Please advise if you think so, and kindly give me some pointers to the documentation on such process (I could not find any information on _selective_ patching in the Porter's Handbook).
Comment 2 Damjan Jovanovic 2016-01-16 13:30:37 UTC
Please scrap that patch, it's fixed upstream now in SVN revision 1724971 (github https://github.com/apache/openoffice/commit/1b37335955a5c2a31c90142ea739bee1bd54a79c). If you want to fix older openoffice versions, backport that patch instead, which will fix encryption functions and digital signatures too, not just cryptographic hash functions.
Comment 3 Arrigo Marchiori 2016-01-19 13:23:15 UTC
Created attachment 165818 [details] Patch including upstream's commit and the proposed patch to enable correct interpretation of the isRelative option Following Damjan's advice, this patch integrates upstream SVN commit 1724971. This patch also includes a proposed fix to a mis-interpretation of Mozilla profile files. This was not accepted by upstream (yet?).
Comment 4 Arrigo Marchiori 2016-01-19 13:35:53 UTC
Created attachment 165819 [details] Integration of upstream SVN commits This contains the following changes: 1- upstream SVN commit 1724971, following Damjan's advice from comment #2; 2- fix to a mis-interpretation of the "IsRelative" option inside Mozilla profile files (this is pending review from upstream); 3- PORTREVISION bump. Parts 1- and 2- are inside a single patch file, to be created into the files/ directory.
Comment 5 Arrigo Marchiori 2016-01-22 07:36:32 UTC
Comment on attachment 165819 [details] Integration of upstream SVN commits Both fixes were merged upstream.
Comment 6 commit-hook 2016-01-29 00:44:31 UTC
A commit references this bug: Author: truckman Date: Fri Jan 29 00:44:00 UTC 2016 New revision: 407439 URL: https://svnweb.freebsd.org/changeset/ports/407439 Log: Fix nss-related problems with password-protected documents. Apply patches from upstream commits r1724971 and r1726068 to fix upstream bug <https://bz.apache.org/ooo/show_bug.cgi?id=125431>, "The Password is incorrect. The file cannot be opened." PR: 206234 Submitted by: Arrigo Marchiori <ardovm AT yahoo.it> MFH: 2106Q1 Changes: head/editors/openoffice-4/Makefile head/editors/openoffice-4/files/patch-nss
Comment 7 commit-hook 2016-01-29 19:33:09 UTC
A commit references this bug: Author: truckman Date: Fri Jan 29 19:32:13 UTC 2016 New revision: 407493 URL: https://svnweb.freebsd.org/changeset/ports/407493 Log: MFH: r405096 (direct commit to openoffice-4 only) MFH: r406872 (direct commit to openoffice-4 only) MFH: r407439 r405096: Correct the Icon entry in the .desktop files so that the icons are visible in the menu. r406872: PORTREVISION bump to allow a clean merge of the next r470439:  Fix nss-related problems with password-protected documents. Apply patches from upstream commits r1724971 and r1726068 to fix upstream bug <https://bz.apache.org/ooo/show_bug.cgi?id=125431>, "The Password is incorrect. The file cannot be opened." PR: 205758 PR: 206234 Submitted by: Arrigo Marchiori <ardovm AT yahoo.it>  Approved by: ports-secteam (miwi - r407439) Approved by: ports-secteam (feld - r405096, r406872) Changes: _U branches/2016Q1/ branches/2016Q1/editors/openoffice-4/Makefile branches/2016Q1/editors/openoffice-4/files/patch-nss
Comment 8 commit-hook 2016-01-30 21:27:25 UTC
A commit references this bug: Author: truckman Date: Sat Jan 30 21:26:57 UTC 2016 New revision: 407553 URL: https://svnweb.freebsd.org/changeset/ports/407553 Log: Upgrade to upstream svn revision 1727463. This fixes the upstream bug <https://bz.apache.org/ooo/show_bug.cgi?id=125431>, "The Password is incorrect. The file cannot be opened." PR: 206234 Changes: head/editors/openoffice-devel/Makefile head/editors/openoffice-devel/distinfo