Pale Moon crashes on ice[*].somafm.com Topic is solved

Talk about code development, features, specific bugs, enhancements, patches, and similar things.
Forum rules
Please keep everything here strictly on-topic.
This board is meant for Pale Moon source code development related subjects only like code snippets, patches, specific bugs, git, the repositories, etc.

This is not for tech support! Please do not post tech support questions in the "Development" board!
Please make sure not to use this board for support questions. Please post issues with specific websites, extensions, etc. in the relevant boards for those topics.

Please keep things on-topic as this forum will be used for reference for Pale Moon development. Expect topics that aren't relevant as such to be moved or deleted.
Aube Bleue

Pale Moon crashes on ice[*].somafm.com

Unread post by Aube Bleue » 2018-01-27, 20:40

Expected result: http://ice.somafm.com/ (be careful!) should redirect to http://somafm.com/.
Actual result: Pale Moon crashes every time.

FYI: tested with a FRESH profile of Pale Moon v27.7.1 on a fully up-to-date Debian 9 "Stretch" 32-bit installation.

JustOff

Re: Pale Moon crashes on ice[*].somafm.com

Unread post by JustOff » 2018-01-27, 20:48

I confirm the crash on Windows in Pale Moon 27.4.0-27.8.0a1, but all is ok in 27.3.0.

UPD: No crash occurs when scripts on this site are disabled using NoScript.

The response is actually xml (?!) sent with "Content-Type: text/xml" header and the source is:

Code: Select all

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
</head>
  <body>
    <script type="text/javascript">
window.location = "http://SomaFM.com/"
</script>
  </body>
</html>
Last edited by JustOff on 2018-01-27, 21:17, edited 4 times in total.

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 35476
Joined: 2011-08-28, 17:27
Location: Motala, SE
Contact:

Re: Pale Moon crashes on ice[*].somafm.com

Unread post by Moonchild » 2018-01-27, 22:47

Thanks for reporting this! Confirmed; this is a null crash on specifically added code for Pale Moon. Simple issue with a simple fix: Issue #1605
Last edited by Moonchild on 2018-01-27, 23:10, edited 2 times in total.
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite

User avatar
jars_
Lunatic
Lunatic
Posts: 397
Joined: 2016-12-27, 00:12

Re: Pale Moon crashes on ice[*].somafm.com

Unread post by jars_ » 2018-01-28, 07:09

Off-topic:
Hi guys.
This bug may be due to this case: There is an extension Showcase and it works in PaleMoon, but it also results in crash of the brazer . It crashes PaleMoon, without errors in the Сonsole at startup or during use.

To get this error is very simple(with installed extension), you either need to change the page scale quickly, or open the developer tools with an attached window and resize the window - it will crash.
Last edited by Moonchild on 2018-01-31, 13:47, edited 1 time in total.
Reason: Keep your posts on-topic, please.

dinosaur
Fanatic
Fanatic
Posts: 165
Joined: 2014-06-03, 09:26
Location: France

Re: Pale Moon crashes on ice[*].somafm.com

Unread post by dinosaur » 2018-01-30, 11:54

I would recommend also adding a check for mOriginalSink nullptr case in nsExpatDriver::WillBuildModel(), around line 1274 of nsExpatDriver.cpp (same scenario).

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 35476
Joined: 2011-08-28, 17:27
Location: Motala, SE
Contact:

Re: Pale Moon crashes on ice[*].somafm.com

Unread post by Moonchild » 2018-01-31, 14:16

dinosaur wrote:I would recommend also adding a check for mOriginalSink nullptr case in nsExpatDriver::WillBuildModel(), around line 1274 of nsExpatDriver.cpp (same scenario).
No, it's not the same scenario. sinks are already checked at the start of the function, there.
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite

Locked