Live data from Hacker News

Opening http://../foo on Android Chrome crashes the browser (Warning: or worse)

bugs.chromium.org

41–50 of 86 posts

Re: Opening http://../foo on Android Chrome crashes the browser (Warning: or worse)

#41
post #36

Earlier quoted context omitted.

But actual crashes are most likely due to memory unsafety; I’d be extremely surprised if the root cause was not found to be in a memory-unsafe language, though I doubt it’ll be in the URL parser.

That's a weak statement: isn't Chrome is almost entirely written in C++? I suppose you mean it's a memory overrun or use are free. You're probably right, then. But in Rust, something like this could happen too, if someone added a panic!, e.g. on parsing a domain name with an empty component, or failure to put the domain name in a certain (external) structure. If the URL is parsed again on start-up, the application wo…

I would guess that various parts of Chrome for Android, especially its UI, might be in a memory-safe language. But yeah, in hindsight my word choice was poor; I should have said a root cause of memory unsafety.

As for Rust, if we’re talking about outright crashes (which on reflection I’m not sure if we actually are because of laxness in the definition of “crash”—I retract my proffered extreme surprise), panicking isn’t that (unless compiled with abort-on-panic or if a destructor panics while unwinding)—panicking is controlled and can (except for the cases already mentioned) be caught.

To be sure, you can still brick the app in the described way with persisted data triggering exceptions or panicking, but there at least can still be a difference.

Re: Opening http://../foo on Android Chrome crashes the browser (Warning: or worse)

#44

Not an issue in Bromite or GrapheneOS' Vanadium chromium fork. edit: correction: it effects both. Incognito tabs aren't affected.

Well, mine did brick on Vanadium. Didn't try Bromite.

Actually it affects both. I was using Incognito to test it out. Turns out it doesn't work in incognito.

Re: Opening http://../foo on Android Chrome crashes the browser (Warning: or worse)

#45
post #27

Earlier quoted context omitted.

Off Topic: I don't get it why, after 10 years of mobile browser development, there still is no "close all tabs at end" option. Every time I visit my parents, I have to clean up their 150+ open chrome android tabs because they don't understand how to properly clean up their browsing sessions. Why on earth would I want to open the same old tabs in the morning, that I already looked at the day before?

Because they are the tabs with your email, facebook, news, and and and.

That’s what pinned tabs are for.

Re: Opening http://../foo on Android Chrome crashes the browser (Warning: or worse)

#47
post #3

Earlier quoted context omitted.

> Note: this is even worse if the URL was opened from an intent. In that case, Chrome can end up completely bricked because upon restart it will immediately attempt to re-open the URL that crashed it. I could not recover from this without having to fully "Clear Storage". Too late for me. Not even from intent. Chrome force stop later, it still tries to load it and immediately crashes.

Off Topic: I don't get it why, after 10 years of mobile browser development, there still is no "close all tabs at end" option. Every time I visit my parents, I have to clean up their 150+ open chrome android tabs because they don't understand how to properly clean up their browsing sessions. Why on earth would I want to open the same old tabs in the morning, that I already looked at the day before?

Because for a large number of people (including me) tabs are also bookmarks. Granted, most of them I’ll never get back to, but it’s a habit that’s difficult to break.

Re: Opening http://../foo on Android Chrome crashes the browser (Warning: or worse)

#48

Is "foo" used to mean any string, or is it literally only "../foo" that crashes it?

Any string, does not apply to brave so not a seemingly a webkit issue

It wouldn't be. Neither Brave nor Chrome use WebKit.

Re: Opening http://../foo on Android Chrome crashes the browser (Warning: or worse)

#50
post #47

Earlier quoted context omitted.

Off Topic: I don't get it why, after 10 years of mobile browser development, there still is no "close all tabs at end" option. Every time I visit my parents, I have to clean up their 150+ open chrome android tabs because they don't understand how to properly clean up their browsing sessions. Why on earth would I want to open the same old tabs in the morning, that I already looked at the day before?

Because for a large number of people (including me) tabs are also bookmarks. Granted, most of them I’ll never get back to, but it’s a habit that’s difficult to break.

Yep, because you can't save all open tabs as bookmarks
Post reply on HN