CoSwitched / rcsjta

Automatically exported from code.google.com/p/rcsjta
0 stars 0 forks source link

NPE in ServiceUtils.isServiceStarted #61

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
In Google PlayStore reports we've noticed NPE crashed listed in the predecessor 
of ServiceUtils.isServiceStarted (i.e. ClientApiUtils.isServiceStarted). As the 
code hasn't changed I would popose some changes:
* change from 
serviceName.getClassName().equals("com.orangelabs.rcs.service.RcsCoreService") 
to 
"com.orangelabs.rcs.service.RcsCoreService".equals(serviceName.getClassName())
* surround with try/catch block

Original issue reported on code.google.com by andreas-...@telekom.de on 8 Sep 2014 at 9:49

GoogleCodeExporter commented 9 years ago
patch proposal

Original comment by andreas-...@telekom.de on 8 Sep 2014 at 11:55

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by lemordan...@gmail.com on 9 Sep 2014 at 7:37

GoogleCodeExporter commented 9 years ago
with commit 59ab1a7

Original comment by andreas-...@telekom.de on 9 Sep 2014 at 8:40