Opened 5 years ago

Closed 4 years ago

Last modified 3 years ago

#13985 closed bug (no change required)

StyledEdit crashes if started by be_roster->Launch

Reported by: donn Owned by: nobody
Priority: normal Milestone:
Component: - General Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All


See attached example. When started with be_roster->Launch("text/plain", refmsg), StyledEdit crashes in BPrivate::HaikuControlLook::_DrawButtonFrame(), an error which prevents it from continuing.

Attachments (1)

testexec.C (667 bytes ) - added by donn 5 years ago.

Download all attachments as: .zip

Change History (7)

by donn, 5 years ago

Attachment: testexec.C added

comment:1 by diver, 5 years ago

Can't reproduce with x86_64. Are you sure you don't have an older StyledEdit binary lying somewhere?

comment:2 by donn, 5 years ago

Excellent, that indeed appears to have been what happened - there were a couple, in the generated/ directories of an old build of the Haiku OS.

I'm sure BRoster::Launch() is the only way anything ever tried to run those copies of StyledEdit. It's quite bizarre that it would look there, for an application to deal with "text/plain". I think it didn't used to, but in any case I guess this behavior could appear somewhat random.

I guess this may be more or less the same bug as #6475 - if queries are going to be employed in ways that matter, like finding an application to execute for a particular task, then they need some way to prioritize the applications that were installed with the user's expectation that they'd be used for that kind of task. Like POSIX execvpe() uses PATH, instead of picking a randomly discovered instance of the specified file name. Along with the general expectation that things work as expected, there can be security problems, when the system unexpected gropes around in your files for an executable file.

comment:3 by korli, 5 years ago

Since the introduction of PM residual executables on other partitions are not found anymore since the HPKGs (haiku.hpkg) are not mounted. This is really more a power user issue. No change required?

comment:4 by donn, 5 years ago

Note that this StyledEdit wasn't a residual installed earlier on another partition. It was never installed, just built and laying around in my files, in the /boot partition. Consider the security implications!

If the assumption is that executables will be installed only in PM space, maybe file-type -> application launches should be limited to PM space. That's less flexible than PATH and could lead to some frustration, but I maybe everyone could live with it, and has a sane rationale.

comment:5 by waddlesplash, 4 years ago

Resolution: no change required
Status: newclosed

comment:6 by nielx, 3 years ago

Milestone: Unscheduled

Remove milestone for tickets with status = closed and resolution != fixed

Note: See TracTickets for help on using tickets.