Return-Path: X-Original-To: apmail-incubator-ooo-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-ooo-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id AF005D0E6 for ; Fri, 29 Jun 2012 09:53:25 +0000 (UTC) Received: (qmail 52625 invoked by uid 500); 29 Jun 2012 09:53:25 -0000 Delivered-To: apmail-incubator-ooo-dev-archive@incubator.apache.org Received: (qmail 52484 invoked by uid 500); 29 Jun 2012 09:53:24 -0000 Mailing-List: contact ooo-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: ooo-dev@incubator.apache.org Delivered-To: mailing list ooo-dev@incubator.apache.org Received: (qmail 52470 invoked by uid 99); 29 Jun 2012 09:53:24 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 29 Jun 2012 09:53:24 +0000 X-ASF-Spam-Status: No, hits=2.4 required=5.0 tests=FREEMAIL_ENVFROM_END_DIGIT,HTML_MESSAGE,RCVD_IN_DNSWL_NONE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of polo8495@hotmail.com designates 65.54.190.213 as permitted sender) Received: from [65.54.190.213] (HELO bay0-omc4-s11.bay0.hotmail.com) (65.54.190.213) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 29 Jun 2012 09:53:16 +0000 Received: from BAY167-W109 ([65.54.190.199]) by bay0-omc4-s11.bay0.hotmail.com with Microsoft SMTPSVC(6.0.3790.4675); Fri, 29 Jun 2012 02:52:56 -0700 Message-ID: Content-Type: multipart/alternative; boundary="_c1a1871a-5953-4694-b49e-d003e18ad5b1_" X-Originating-IP: [202.108.130.138] From: YangTerry To: Subject: RE: Propose for 3.4.1: Can't remove password from file (119366) Date: Fri, 29 Jun 2012 17:52:56 +0800 Importance: Normal In-Reply-To: <4FEC21F6.6000003@googlemail.com> References: ,<4FE080DB.5050406@googlemail.com>,,<4FE84748.2080508@googlemail.com>,,<4FE84EFC.3020000@googlemail.com>,,<4FE9A57B.9040300@googlemail.com>,<4FE9C6EC.1070804@googlemail.com>,,,,,,,,,<4FEC1F1A.6020306@googlemail.com>,<4FEC21F6.6000003@googlemail.com> MIME-Version: 1.0 X-OriginalArrivalTime: 29 Jun 2012 09:52:56.0474 (UTC) FILETIME=[F5919FA0:01CD55DC] X-Virus-Checked: Checked by ClamAV on apache.org --_c1a1871a-5953-4694-b49e-d003e18ad5b1_ Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: 8bit Test again on trunk r1355082 which include modules nss and moz. >From the result I think we can close this bug. Test with 3 scenario. A. Saved with password, then remove password with same revision B. Saved with password with OOo3.3, then remove password with r1355082 C. Saved with password with MS 2003, then remove password with r1355082 Result: Scenario A: ods/xls/odt/odp work fine doc can saved with password successfully, but when reopen the saved file, it is under read-only mode. This also repro on OOo 3.3 ppt is weird, in OOo 3.3, In saved dialog, the password check box can check, input password and saved the file then reopen it, No need password, so saved with password in OOo3.3 for ppt format is not work. In trunk r1355082, In saved dialog, "saveing with password" checkbox is disabled If we saved ppt file with password protect by OpenOffice, then open in MS Office, no need input password, so it is totally not work. Scenario B: ods/xls/odt/odp work fine doc file with password open under read-only mode, we can saved to another file to remove password. ppt file can't saved with password protect Scenario C: xls work fine doc file with password open under read-only mode, we can saved to another file to remove password. ppt file with password protect by MS office can't open in Open Office, it said "Read Error. the loading of password-encrypted Microsoft PowerPint presentations is not supported." For PPT issue, there have 2 bugs about saved(Bug 39527) and loading(Bug 46307) For doc open with read-only mode issue, i think it is a know issue, but i can't find the bug in bugzilla, I will double check in bugzilla. > Date: Thu, 28 Jun 2012 11:20:54 +0200 > From: orwittmann@googlemail.com > To: ooo-dev@incubator.apache.org > Subject: Re: Propose for 3.4.1: Can't remove password from file (119366) > > Hi, > > On 28.06.2012 11:08, Oliver-Rainer Wittmann wrote: > > Hi, > > > > sorry for top posting - it is reply to the complete discussion regarding "save > > with password" > > > > This functionality for ODF documents is available when the build includes > > modules nss and moz. > > > > I am not sure, if our buildbots include these modules. I will check it. > > > > win7 buildbot includes nss and moz > linux32 buildbot includes nss and moz > linux64 buildbot includes nss and moz > > But, as you may have notice since a couple of days our buildbots have problems > to create new packages. > last sucessful build for win7 was on 2012-06-19 > last sucessful build for linux32 was on 2012-06-20 > last sucessful build for linxu64 was on 2012-06-17 > none of these contains the fix for issue 119366 > > Best regards, Oliver. > > > > > Best regards, Oliver. > > > > On 28.06.2012 10:21, YangTerry wrote: > >> > >> Just confirm with our build owner, moz package is not in the build. > >> Will verify and update the result after download the trunk build from BuildBot. > >> > >> Thanks Fan Zheng help (*^__^*) > >> > >> > >>> From: polo8495@hotmail.com > >>> To: ooo-dev@incubator.apache.org > >>> Subject: RE: Propose for 3.4.1: Can't remove password from file (119366) > >>> Date: Thu, 28 Jun 2012 16:02:31 +0800 > >>> > >>> > >>> Will confirm with our local build owner. Also download the trunk build from http://ci.apache.org/projects/openoffice/ > >>> > >>> Thanks for your investgate. > >>> > >>>> Date: Thu, 28 Jun 2012 15:19:39 +0800 > >>>> Subject: Re: Propose for 3.4.1: Can't remove password from file (119366) > >>>> From: zheng.easyfan@gmail.com > >>>> To: ooo-dev@incubator.apache.org > >>>> > >>>> Hold a second, you guys are using the download build on verifying, right? > >>>> > >>>> If so, that means some problems there. Maybe the release build env broken? > >>>> > >>>> > >>>> 2012/6/28 Fan Zheng > >>>> > >>>>> Muhaha, lucky for me that I have 3 build enviroment on verification this, > >>>>> all of them are under Windows XP. The difference are: > >>>>> In build env A, there is no moz package involved; > >>>>> In build env B, there is moz package involved, but without my solution of > >>>>> issue 119366; > >>>>> In build env C, there is moz package involved and with my solution of > >>>>> issue 119366; > >>> > >>>>> And I did the following test cases: > >>>>> In env A: > >>>>> 1.1 Save into ODT, without password, passed; > >>>>> 1.2 Save into ODT, with password, failed, with error message "General > >>>>> Error: Generral input/output error"; > >>>>> 1.3 Save into DOC, without password, passed; > >>>>> 1.4 Save into DOC, with password, passed; But in continual case: "Save As > >>>>> without password", failed; > >>>>> 2.1 Save into ODP, without password, passed; > >>>>> 2.2 Save into ODP, with password, failed, with error message "General > >>>>> Error: Generral input/output error"; > >>>>> 2.3 Save into PPT, without password, passed; > >>>>> 2.4 Save into PPT, the "saveing with password" checkbox is disabled... > >>>>> Weird. > >>>>> 3.1 Save into ODS, without password, passed; > >>>>> 3.2 Save into ODS, with password, failed, with error message "General > >>>>> Error: Generral input/output error"; > >>>>> 3.3 Save into XLS, without password, passed; > >>>>> 3.4 Save into XLS, with password, passed; But in continual case: "Save As > >>>>> without password", failed; > >>>>> > >>>>> In env B: > >>>>> 1.1 Save into ODT, without password, passed; > >>>>> 1.2 Save into ODT, with password, passed; But in continual case: "Save As > >>>>> without password", failed; > >>>>> 1.3 Save into DOC, without password, passed; > >>>>> 1.4 Save into DOC, with password, passed; But in continual case: "Save As > >>>>> without password", failed; > >>>>> 2.1 Save into ODP, without password, passed; > >>>>> 2.2 Save into ODP,with password, passed; But in continual case: "Save As > >>>>> without password", failed; > >>>>> 2.3 Save into PPT, without password, passed; > >>>>> 2.4 Save into PPT, the "saveing with password" checkbox is disabled... > >>>>> Weird. > >>>>> 3.1 Save into ODS, without password, passed; > >>>>> 3.2 Save into ODS, with password, passed; But in continual case: "Save As > >>>>> without password", failed; > >>>>> 3.3 Save into XLS, without password, passed; > >>>>> 3.4 Save into XLS, with password, passed; But in continual case: "Save As > >>>>> without password", failed; > >>>>> > >>>>> In env C: > >>>>> 1.1 Save into ODT, without password, passed; > >>>>> 1.2 Save into ODT, with password, passed; In continual case: "Save As > >>>>> without password", Passed; > >>>>> 1.3 Save into DOC, without password, passed; > >>>>> 1.4 Save into DOC, with password, passed; In continual case: "Save As > >>>>> without password", Passed; > >>>>> 2.1 Save into ODP, without password, passed; > >>>>> 2.2 Save into ODP,with password, passed; In continual case: "Save As > >>>>> without password", Passed; > >>>>> 2.3 Save into PPT, without password, passed; > >>>>> 2.4 Save into PPT, the "saveing with password" checkbox is disabled... > >>>>> Weird. > >>>>> 3.1 Save into ODS, without password, passed; > >>>>> 3.2 Save into ODS, with password, passed; In continual case: "Save As > >>>>> without password", Passed; > >>>>> 3.3 Save into XLS, without password, passed; > >>>>> 3.4 Save into XLS, with password, passed; In continual case: "Save As > >>>>> without password", Passed; > >>>>> > >>>>> Conclution: > >>>>> > >>>>> Such error messages are caused missing moz package building involved. > >>>>> And such I/O error message only occurs in the ODF format file saving with > >>>>> password. > >>>>> Both ODF format saving with password and OFFICE format saving with > >>>>> password have the issue 119366; > >>>>> My solution could work on both MS office files saving and ODF format > >>>>> saving; > >>>>> > >>>>> For you review. > >>>>> > >>>>> > >>>>> 2012/6/28 YangTerry > >>>>> > >>>>>> > >>>>>> I can saved .ods file without password successfully, also can save .xls > >>>>>> successfully. > >>>>>> > >>>>>> Only when i saved our format (like .ods) file with password, this error > >>>>>> pop up. > >>>>>> > >>>>>> Also failed to open our format file with password. > >>>>>> > >>>>>> > >>>>>> > >>>>>> So it should not be related with build without moz package. > >>>>>> > >>>>>> > >>>>>>> Date: Thu, 28 Jun 2012 13:57:14 +0800 > >>>>>>> Subject: Re: Propose for 3.4.1: Can't remove password from file (119366) > >>>>>>> From: zheng.easyfan@gmail.com > >>>>>>> To: ooo-dev@incubator.apache.org > >>>>>>> > >>>>>>> And I my local, (WinXP env), seems such issues can not be reproduced. > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>> 2012/6/28 Fan Zheng > >>>>>>> > >>>>>>>> The error message "General Error: Generral input/output error" looks > >>>>>> so > >>>>>>>> farmilliar... I remember that in the build without moz package > >>>>>> inside, such > >>>>>>>> dialog will show up. > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> 2012/6/28 YangTerry > >>>>>>>> > >>>>>>>>> > >>>>>>>>> Verify not fixed on trunk r1354384. > >>>>>>>>> If we saved with our format(.ods) > >>>>>>>>> Failed to saved with password, the failed message in dialog is > >>>>>> "General > >>>>>>>>> Error: Generral input/output error", also failed open the password > >>>>>> protect > >>>>>>>>> .ods file, it pop up password incorrect dialog but i input correct > >>>>>>>>> password. Reopen this bug. > >>>>>>>>> My Platform is Win 7 X64 EN.If we saved with MS format (.xls), > >>>>>>>>> successfully saved it with password and also work fine to removed the > >>>>>>>>> password. > >>>>>>>>> Also work fine saved file(.ods) without password. > >>>>>>>>> > >>>>>>>>> Seems something wrong in our format save with password logic. > >>>>>>>>> > >>>>>>>>>> Date: Tue, 26 Jun 2012 16:27:56 +0200 > >>>>>>>>>> From: orwittmann@googlemail.com > >>>>>>>>>> To: ooo-dev@incubator.apache.org > >>>>>>>>>> Subject: Re: Propose for 3.4.1: Can't remove password from file > >>>>>> (119366) > >>>>>>>>>> > >>>>>>>>>> Hi, > >>>>>>>>>> > >>>>>>>>>> On 26.06.2012 14:05, Oliver-Rainer Wittmann wrote: > >>>>>>>>>>> Hi, > >>>>>>>>>>> > >>>>>>>>>>> On 26.06.2012 09:53, Fan Zheng wrote: > >>>>>>>>>>>> Root cause: > >>>>>>>>>>>> > >>>>>>>>>>>> Seems the logic of "Save As" and "Save" inside Apache > >>>>>> OpenOffice is > >>>>>>>>> pretty > >>>>>>>>>>>> weird anyway. > >>>>>>>>>>>> A, inside AOO, the method SfxBaseModel::StoreSelf is the entry > >>>>>> for > >>>>>>>>> storing > >>>>>>>>>>>> file into the original URL path if it has one. Which means, such > >>>>>>>>> method is > >>>>>>>>>>>> responsible to: > >>>>>>>>>>>> > >>>>>>>>>>>> 1. Directly "Save" request, but exclude the very first time on > >>>>>> "Save" > >>>>>>>>>>>> without original URL path; > >>>>>>>>>>>> > >>>>>>>>>>>> 2. "SaveAs" request, with the same URL information as former; > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> B, as such method is only focus on storing back into to original > >>>>>>>>> file, it > >>>>>>>>>>>> is designed as an incremental saving pattern for certain > >>>>>> efficient > >>>>>>>>>>>> consideration. Which means, > >>>>>>>>>>>> such function do not allow external saving parameters except the > >>>>>>>>> ones on > >>>>>>>>>>>> changing "Version Comments", "Author", "Interaction Handler" and > >>>>>>>>> "status > >>>>>>>>>>>> Indicator". > >>>>>>>>>>>> > >>>>>>>>>>>> C, "Saving with password" is a kind of external saving > >>>>>> parameter. The > >>>>>>>>>>>> saving parameters set will contain a password item inside, if > >>>>>> users > >>>>>>>>> have > >>>>>>>>>>>> enable the check box > >>>>>>>>>>>> "Save with password" in "File Save As" dialog. Otherwise, saving > >>>>>>>>> parameters > >>>>>>>>>>>> set wont contain password corresponding items. > >>>>>>>>>>>> > >>>>>>>>>>>> Combine the above 3 conditions, we can take a deeper inside > >>>>>> look of > >>>>>>>>>>>> following scenarios: > >>>>>>>>>>>> > >>>>>>>>>>>> 1. In the "Save" request, whatever the password originally > >>>>>> enabled > >>>>>>>>> or not, > >>>>>>>>>>>> as no further different setting applied, the storing process > >>>>>> will > >>>>>>>>> directly > >>>>>>>>>>>> apply the former saving parameters set, including the URL path > >>>>>> and > >>>>>>>>> password > >>>>>>>>>>>> setting stuff. Everything is OK. > >>>>>>>>>>>> > >>>>>>>>>>>> 2. And in the "SaveAs" request with password originally > >>>>>> disabled: > >>>>>>>>>>>> 2.1 If the user keep the "Save with password" disabled in "File > >>>>>> Save > >>>>>>>>> As" > >>>>>>>>>>>> dialog, as no further setting applied, the storing process will > >>>>>>>>> directly > >>>>>>>>>>>> apply the former saving parameters set, still with password > >>>>>>>>> disabled. Keep > >>>>>>>>>>>> the consistence between UI setting and exact result and high > >>>>>>>>> efficiency; > >>>>>>>>>>>> 2.2 If the user change the "Save with password" from disable to > >>>>>>>>> enable in > >>>>>>>>>>>> "File Save As" dialog, as external saving parameter was added > >>>>>> into > >>>>>>>>> saving > >>>>>>>>>>>> parameters set, which do not satisfy the verification of > >>>>>> parameters, > >>>>>>>>> such > >>>>>>>>>>>> "SaveAs" request will be returned from SfxBaseModel::StoreSelf, > >>>>>> and > >>>>>>>>>>>> actually finished inside the common "SaveAs" method with > >>>>>> password > >>>>>>>>> enabled. > >>>>>>>>>>>> Also keep the consistence between UI setting and exact result; > >>>>>>>>>>>> 3. In the "SaveAs" request with password originally enabled: > >>>>>>>>>>>> 3.1 If the user keep the "Save with password" enabled in "File > >>>>>> Save > >>>>>>>>> As" > >>>>>>>>>>>> dialog, as external saving parameter was added into saving > >>>>>>>>> parameters set, > >>>>>>>>>>>> which do not satisfy the verification of parameters, such > >>>>>> "SaveAs" > >>>>>>>>> request > >>>>>>>>>>>> will be returned from SfxBaseModel::StoreSelf, and actually > >>>>>> finished > >>>>>>>>> inside > >>>>>>>>>>>> the common "SaveAs" method with password enabled. Keep the > >>>>>>>>> consistence > >>>>>>>>>>>> between UI setting and exact result, but with lower efficiency; > >>>>>>>>>>>> 3.2 If the user change the "Save with password" from enabled to > >>>>>>>>> disabled in > >>>>>>>>>>>> "File Save As" dialog, as no further setting applied, the > >>>>>> storing > >>>>>>>>> process > >>>>>>>>>>>> will directly apply the former saving parameters set, still with > >>>>>>>>> password > >>>>>>>>>>>> enabled, as oppose to the UI setting. The issue happens. > >>>>>>>>>>>> > >>>>>>>>>>>> So, a reasonable solution of this issue should be: > >>>>>>>>>>>> > >>>>>>>>>>>> 1. No process and saving parameter change on scenario 1 and 2; > >>>>>>>>>>>> 2. In scenario 3.1, remove the external password parameter as > >>>>>> the > >>>>>>>>>>>> originally enabled, and makes it finished in StoreSelf for > >>>>>> higher > >>>>>>>>>>>> efficiency; > >>>>>>>>>>>> 3. In scenario 3.2, do not trying to use StoreSelf anyway; > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> For you reference. > >>>>>>>>>>>> > >>>>>>>>>>>> The code patch will be submitted for reviewing later. > >>>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> Thanks for this really deep and well founded analysis. > >>>>>>>>>>> > >>>>>>>>>>> I am currently reviewing the new patch. > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> patch looks good - I will commit it to trunk and branch AOO34 soon. > >>>>>>>>>> Thx ZhengFan. > >>>>>>>>>> > >>>>>>>>>> Best regards, Oliver. > >>>>>>>>> > >>>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>> > >>>>>> > >>>>> > >>>>> > >>> > >> > >> > > > --_c1a1871a-5953-4694-b49e-d003e18ad5b1_--