[haiku-bugs] [Haiku] #11582: BRoster::Launch() mistakenly returns B_OK

  • From: "ttcoder" <trac@xxxxxxxxxxxx>
  • Date: Mon, 08 Dec 2014 15:39:34 -0000

#11582: BRoster::Launch() mistakenly returns B_OK
----------------------------------+------------------------------
 Reporter:  ttcoder               |        Owner:  axeld
     Type:  bug                   |       Status:  new
 Priority:  normal                |    Milestone:  R1
Component:  Kits/Application Kit  |      Version:  R1/Development
 Keywords:                        |   Blocked By:
 Blocking:                        |  Has a Patch:  0
 Platform:  All                   |
----------------------------------+------------------------------
 This returns B_OK:

 {{{
 entry_ref bogus;
 be_roster->Launch(&bogus);
 }}}

 And so does this:
 {{{
 entry_ref ref;
 get_ref_for_path(&ref, path to a non-existing filename in an existing
 folder);
 be_roster->Launch(&ref);
 }}}

 Ran into that while hacking media_server trying to [http://cgit.haiku-
 os.org/haiku/tree/src/servers/media/media_server.cpp#n206 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).

--
Ticket URL: <https://dev.haiku-os.org/ticket/11582>
Haiku <https://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: