Commit acedadfa authored by Dan Carpenter's avatar Dan Carpenter Committed by Greg Kroah-Hartman
Browse files

Staging: bcm: dereferencing before checking 



I moved the check to see if "Adapter" was null in front of the
dereference.

Signed-off-by: default avatarDan Carpenter <error27@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 055a1eda

drivers/staging/bcm/CmHost.c

+5 −6
Original line number Diff line number Diff line
@@ -1943,15 +1943,14 @@ ULONG SetUpTargetDsxBuffers(PMINI_ADAPTER Adapter) 
	ULONG ulIndex=0;

	int Status;



	if(Adapter->astTargetDsxBuffer[0].ulTargetDsxBuffer)

		return 1;



	 if(NULL == Adapter)

	 {

	if (!Adapter) {

		BCM_DEBUG_PRINT(Adapter, DBG_TYPE_OTHERS, CONN_MSG, DBG_LVL_ALL, "Adapter was NULL!!!");

		return 0;

	}



	if(Adapter->astTargetDsxBuffer[0].ulTargetDsxBuffer)

		return 1;



	BCM_DEBUG_PRINT( Adapter,DBG_TYPE_OTHERS, CONN_MSG, DBG_LVL_ALL, "Size of Each DSX Buffer(Also size of ServiceFlowParamSI): %zx ",sizeof(stServiceFlowParamSI));

	BCM_DEBUG_PRINT( Adapter,DBG_TYPE_OTHERS, CONN_MSG, DBG_LVL_ALL, "Reading DSX buffer From Target location %x ",DSX_MESSAGE_EXCHANGE_BUFFER);