From c3f16206d77043f15dee31c6dec6a1d38fdb6ae2 Mon Sep 17 00:00:00 2001
From: Tushar Janefalkar <tusharj@codeaurora.org>
Date: Tue, 18 Jun 2013 11:06:27 -0700
Subject: [PATCH] Return the correct value from loc_eng_init

The function was returning an error code
which was causing GPSLocationProvider to
fail initialization

CRs-fixed: 501318
Change-Id: Icec707c3b9edc60bd07914b130077f9a96c5e794
---
 loc_api/libloc_api_50001/loc_eng.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/loc_api/libloc_api_50001/loc_eng.cpp b/loc_api/libloc_api_50001/loc_eng.cpp
index 21cee3e0..17d2cbe6 100644
--- a/loc_api/libloc_api_50001/loc_eng.cpp
+++ b/loc_api/libloc_api_50001/loc_eng.cpp
@@ -290,11 +290,12 @@ int loc_eng_init(loc_eng_data_s_type &loc_eng_data, LocCallbacks* callbacks,
                   void (*loc_external_msg_sender) (void*, void*))
 
 {
-    int ret_val =-1;
+    int ret_val = 0;
 
     ENTRY_LOG_CALLFLOW();
     if (NULL == callbacks || 0 == event) {
         LOC_LOGE("loc_eng_init: bad parameters cb %p eMask %d", callbacks, event);
+        ret_val = -1;
         EXIT_LOG(%d, ret_val);
         return ret_val;
     }
@@ -1931,13 +1932,14 @@ static void loc_eng_deferred_action_thread(void* arg)
             }
             else
                 LOC_LOGE("Ulp Phone context request call back not initialized");
-            }
+        }
         break;
 
         case LOC_ENG_MSG_LOC_INIT:
         {
             loc_eng_reinit(*loc_eng_data_p);
         }
+        break;
 
         default:
             LOC_LOGE("unsupported msgid = %d\n", msg->msgid);