Crash for profiles with a non-existent non-relative profile location

RESOLVED FIXED in mozilla1.9.3a1



Startup and Profile System
8 years ago
8 years ago


(Reporter: Gabri, Assigned: Gabri)



Firefox Tracking Flags

(status1.9.2 beta5-fixed)



(1 attachment)



8 years ago
Steps to reproduce:

- Download and unzip a trunk build on your desktop
- Make a batch file with ("profile" folder must not exist):
  start C:\......\firefox\firefox.exe -profile profile

Message returned: "Firefox is already running, but is not responding. To open a new window, you must first close the existing Firefox process, or restart your system.".

Comment 1

8 years ago
It should create the "profile" folder automatically or display to the user if he wants to create it.

Comment 2

8 years ago
Created attachment 414916 [details] [diff] [review]
proposed patch

It works, I hope benjamin could review it.
Attachment #414916 - Flags: review?(benjamin)
Attachment #414916 - Flags: review?(benjamin) → review+
Assignee: nobody →
Component: Profile: BackEnd → Startup and Profile System
Keywords: checkin-needed
Product: Core → Toolkit
QA Contact: profile-manager-backend → startup
Dup of bug 278860?
No, not at all.
(In reply to comment #4)
> No, not at all.

Benjamin Smedberg, what is difference of this bug from problem of bug 278860?
Proposed patch to this bug never covers problem of bug 278860?
Or you saying that "Firefox is already running, but is not responding. ..." when non-relative path(this bug) is absolutely different problem from the message/phenomenon when relative path(bug 278860)?
Or this bug is really for crash even though Crash-ID is not reported to this bug?

Comment 6

8 years ago
This bug is specifically about -profile so it's not a dup of bug 278860.
I'll take a look to bug 278860 soon :)


8 years ago
Attachment #414916 - Flags: approval1.9.2?
Comment on attachment 414916 [details] [diff] [review]
proposed patch

Attachment #414916 - Flags: approval1.9.2? → approval1.9.2+
(In reply to comment #6)
> I'll take a look to bug 278860 soon :)

Gabriele Best, I hope you'll resolve bug 459638 too, which Benjamin closed as WONTFIX once in the past :-)
(Note: I understand why Benjyamin closed that bug as WONTFIX once, but I can't accept WONTFIX of that bug, although I can explain "why Bug 491295 was WONTFIXE'ed/it's reasonable/we should acceptt it" to users and bug openers for similar phenomenon to Bug 491295.)
Keywords: checkin-needed
Target Milestone: --- → mozilla1.9.3a1
Last Resolved: 8 years ago
Resolution: --- → FIXED


8 years ago
Keywords: checkin-needed
status1.9.2: --- → final-fixed
Keywords: checkin-needed


8 years ago
Target Milestone: mozilla1.9.3a1 → mozilla1.9.2
target should stay to the first branch where the patch has been pushed.
Target Milestone: mozilla1.9.2 → mozilla1.9.3a1
You need to log in before you can comment on or make changes to this bug.