Opened 15 months ago

Closed 4 days ago

#13985 closed bug (no change required)

StyledEdit crashes if started by be_roster->Launch

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

Description

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 15 months ago.

Download all attachments as: .zip

Change History (6)

Changed 15 months ago by donn

Attachment: testexec.C added

comment:1 Changed 15 months ago by diver

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

comment:2 Changed 15 months ago by donn

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 Changed 15 months ago by korli

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 Changed 15 months ago by donn

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 Changed 4 days ago by waddlesplash

Resolution: no change required
Status: newclosed
Note: See TracTickets for help on using tickets.