[Openhpi-svn] SF.net SVN: openhpi:[7535] openhpi/trunk/plugins/oa_soap/ oa_soap_re_discover.c

[<hem...@us...>]

Revision: 7535
          http://openhpi.svn.sourceforge.net/openhpi/?rev=7535&view=rev
Author:   hemanthreddy
Date:     2013-05-21 22:28:02 +0000 (Tue, 21 May 2013)
Log Message:
-----------
Fix for issue ID:3613536 - oa_soap_re_discover_resources() arguments checks need fixing

Modified Paths:
--------------
    openhpi/trunk/plugins/oa_soap/oa_soap_re_discover.c

Modified: openhpi/trunk/plugins/oa_soap/oa_soap_re_discover.c
===================================================================
--- openhpi/trunk/plugins/oa_soap/oa_soap_re_discover.c	2013-05-21 21:05:25 UTC (rev 7534)
+++ openhpi/trunk/plugins/oa_soap/oa_soap_re_discover.c	2013-05-21 22:28:02 UTC (rev 7535)
@@ -130,8 +130,6 @@
         SaErrorT rv = SA_OK;
         struct oa_soap_handler *oa_handler = NULL;
 
-        oa_handler = (struct oa_soap_handler *) oh_handler->data;
-
         if (oh_handler == NULL || oa == NULL) {
                 err("Invalid parameters");
                 return SA_ERR_HPI_INVALID_PARAMS;
@@ -139,6 +137,8 @@
 
         err("Re-discovery started");
 
+        oa_handler = (struct oa_soap_handler *) oh_handler->data;
+
 	/* The following is applicable only to OA Switchover cases
 	Just rediscover oa and end the whole thing. If some other hardware
 	is removed at the same time, then it is a separate case that needs to

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.