If I put in Search Term or Firefox Theme URL: https://addons.mozilla.org/ LO will crash. Same if I add ../en-US/ Note: I had a problem with firebird during build process so I had to install on my system libtommath-1.0-1 to build it separately.
On pc Debian x86-64 with master sources updated yesterday, I could reproduce this. I don't reproduce this with LO Debian package 5.2.0.2 => regression
Created attachment 126928 [details] bt with symbols
Regression from https://cgit.freedesktop.org/libreoffice/core/commit/?id=3f22f287b4ae80c5879f87071be93c3ea5d1001c
Created attachment 126929 [details] bt with symbols for ../en-US/ Here's another bt completely different with the other url.
Julien Nabet committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=454e1ebcfc589ce82c8c2d5585e22bea1ea5fc79 tdf#101629: fix https://addons.mozilla.org/ case It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
(In reply to Commit Notification from comment #5) > Julien Nabet committed a patch related to this issue. > It has been pushed to "master": > > http://cgit.freedesktop.org/libreoffice/core/commit/ > ?id=454e1ebcfc589ce82c8c2d5585e22bea1ea5fc79 > > tdf#101629: fix https://addons.mozilla.org/ case > > It will be available in 5.3.0. > > The patch should be included in the daily builds available at > http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More > information about daily builds can be found at: > http://wiki.documentfoundation.org/Testing_Daily_Builds > > Affected users are encouraged to test the fix and report feedback. I ran git pull -r and then make again. If I test again https://addons.mozilla.org/ it works now. I can see themes. Same if I put a tag like Abstract etc...it works fine. Thank you. If you try however https://addons.mozilla.org/en-US/ well the result is weird: "Cannot open https://services.addons.mozilla.org/en-US/firefox/api/1.5/search/https://addons.mozilla.org/en-US//9/15, pleae try again alter.". No crash. I think nobody will care but I am reporting this for info.
@Julien: unfortunately the fix was the right thing to do. It should have checked if it is a url and if it is an invalid theme url, it should stop the search and show an error message dialog, as the current fix will search the themes for that invalid url, as can be seen in comment 6.
(In reply to Yousuf Philips (jay) from comment #7) > @Julien: unfortunately the fix was the right thing to do. It should have > checked if it is a url and if it is an invalid theme url, it should stop the > search and show an error message dialog, as the current fix will search the > themes for that invalid url, as can be seen in comment 6. I suppose you meant, "wasn't the right thing to do". In this case, I'll revert it ASAP.
(In reply to Julien Nabet from comment #8) > I suppose you meant, "wasn't the right thing to do". In this case, I'll > revert it ASAP. Yes forgot the not part :D. If you could improve on your patch that would be fine, else revert.
(In reply to Yousuf Philips (jay) from comment #9) > (In reply to Julien Nabet from comment #8) > > I suppose you meant, "wasn't the right thing to do". In this case, I'll > > revert it ASAP. > > Yes forgot the not part :D. If you could improve on your patch that would be > fine, else revert. Here's the revert commit for review: https://gerrit.libreoffice.org/#/c/28605/
Julien Nabet committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=a8a7b7114d96b2013b00e9bc0ced81aad5495457 Revert "tdf#101629: fix https://addons.mozilla.org/ case" It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
The patch is reverted now.
The fixed should be something like this if ( searchTerm.startsWith( "https://" ) OR searchTerm.startsWith( "http://" ) ) { if ( searchTerm.startsWith( "https://addons.mozilla.org/en-US/firefox/addon/" ) { { download and install searchTerm theme } } else { { show error message and return to dialog } } } else { { strip away unnecessary characters from searchTerm and then query } }
** Please read this message in its entirety before responding ** To make sure we're focusing on the bugs that affect our users today, LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed bugs which have not been touched for over a year. There have been thousands of bug fixes and commits since anyone checked on this bug report. During that time, it's possible that the bug has been fixed, or the details of the problem have changed. We'd really appreciate your help in getting confirmation that the bug is still present. If you have time, please do the following: Test to see if the bug is still present on a currently supported version of LibreOffice (5.4.1 or 5.3.6 https://www.libreoffice.org/download/ If the bug is present, please leave a comment that includes the version of LibreOffice and your operating system, and any changes you see in the bug behavior If the bug is NOT present, please set the bug's Status field to RESOLVED-WORKSFORME and leave a short comment that includes your version of LibreOffice and Operating System Please DO NOT Update the version field Reply via email (please reply directly on the bug tracker) Set the bug's Status field to RESOLVED - FIXED (this status has a particular meaning that is not appropriate in this case) If you want to do more to help you can test to see if your issue is a REGRESSION. To do so: 1. Download and install oldest version of LibreOffice (usually 3.3 unless your bug pertains to a feature added after 3.3) http://downloadarchive.documentfoundation.org/libreoffice/old/ 2. Test your bug 3. Leave a comment with your results. 4a. If the bug was present with 3.3 - set version to "inherited from OOo"; 4b. If the bug was not present in 3.3 - add "regression" to keyword Feel free to come ask questions or to say hello in our QA chat: http://webchat.freenode.net/?channels=libreoffice-qa Thank you for helping us make LibreOffice even better for everyone! Warm Regards, QA Team MassPing-UntouchedBug-20170929
Tried to install Firefox theme with the following url https://addons.mozilla.org/en-US/firefox/addon/brushed-like-chrome/?src=rating and it keeps crashing. LO 5.4.5 on Mac Osx 10.13.3 High Sierra
I submitted a new patch for review here: https://gerrit.libreoffice.org/#/c/51607/ I tried to apply https://bugs.documentfoundation.org/show_bug.cgi?id=101629#c13 except I don't know which unecessary characters to remove. Also what about this url: https://services.addons.mozilla.org/en-US/firefox/api/1.5/search/ ?
Muhammet Kara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=e59ea346d31d416e61b963648e78c5c7f13d8551 tdf#101629: Handle invalid urls and search strings for Personas It will be available in 6.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
I've had been looking forward to the this fix and I installed 6.2.0 on my Windows 10 Pro 64bit OS version 1809. I would like to say the good news is that it looks like it now handles invalid urls and doesn't crash, however I thought this fix would more crucially deal with handling valid URL's, which I tested first, maybe this was overlooked? because if you do a search for firefox themes in AskLibreOffice there are several questions/issues relating to this problem going back to Sept 2015 and the problem was NOT really that LO didn't handle invalid URL's,so much, who really cares about that, except that you don't want it to crash? what is way more important than that is that LO handles VALID URL's, allowing users to actually view and load the Firefox themes, which if you look at all those posts on AskLibreOffice you will see that there is more of a problem with valid URL's since the problem has been reported and practically spelt out many times and those posts have been viewed hundreds of times, so lots of users interested, including myself for whom its extremely important to be able to choose a darker theme to preserve my ever decreasing eyesight, which has been depleted by having to look at insanely bright white backgrounds for far too long including the one I am typing in now :( but after this fix I installed LO 6.2.0 and I entered the actual URL of the location where firefox themes are located namely: https://addons.mozilla.org/en-US/firefox/themes/ and I get the error "Error "Please enter a valid theme address" This functionality is unclear anyway and needs clarification of how its intended to work? Intuitively though from looking at this dialog, you would expect to simply enter the URL to the firefox themes as above, since under the url field there is a drop-down containing all of the Firefox theme sub-categories as per the website, so you wouldn't really expect that you would need to provide a specific theme's URL would you? well yes being a tester, you try everything and leave no stone unturned and users are testers of a kind, like I tried a precise url to a single theme as follows: https://addons.mozilla.org/en-US/firefox/addon/matte-black-v2/?src=hotness This crashed LO Writer and it reported: Due to an unexpected error LO crashed... Does anyone actually test these bugfixes before they are released? If this is to be implemented and fixed properly then the fix needs to consider and fully test, both valid and invalid URL's and LO really need to review how this is implemented! In my humble opinion, when I open this dialog and choose "Own Theme" the url to the firefox themes (https://addons.mozilla.org/en-US/firefox/themes/) should be embedded with this URL as the default, LO should not expect the end user to provide the URL, that is unless for some reason Mozilla have changed the URL and you need to modify it then that would make sense. The only consolation for LO users waiting on this fix is you can now also choose and apply any of the LO presets :)
(In reply to stevejb33 from comment #18) >... LO isn't responsible about Mozilla's changes. See https://bugs.documentfoundation.org/show_bug.cgi?id=123228