Opened 4 years ago

Last modified 3 months ago

#11582 new bug

BRoster::Launch() mistakenly returns B_OK

Reported by: ttcoder Owned by: axeld
Priority: normal Milestone: R1
Component: Kits/Application Kit Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All


This returns B_OK:

entry_ref bogus;

And so does this:

entry_ref ref;
get_ref_for_path(&ref, path to a non-existing filename in an existing folder);

Ran into that while hacking media_server trying to make it launch a custom/debug build of media_addon_server.

At a quick glance, it could be because BRoster::_TranslateRef() mistakenly believes that calling entry.SetTo(ref, false); on a non-existing name will return an error (just a quick guess).

Change History (2)

comment:1 Changed 4 years ago by pulkomandy

I'm not sure, it seems _TranslateRef would fail a few lines later on creating a BNode from the entry if that was the problem?

comment:2 Changed 3 months ago by ttcoder

(note to self: BRoster::_LaunchApp() was tweaked in hrev52968)

Note: See TracTickets for help on using tickets.