First, this function always immediately calls the ofono callback.  That seems wrong, unless there's a specific reason to do so.  Also, in other part of the code when the cb is invoked right away, it's usually best to do via an idle add.

Second, this is a really slopped way to set ofono_error.  CALLBACK_WITH_SUCCESS would be much better!