From: Phil Carmody <ext-phil.2.carmody@xxxxxxxxx> There's no point in continuing if zlLib is null, so just set the status to an error value, and let the rest of the code be skipped over naturally. Signed-off-by: Phil Carmody <ext-phil.2.carmody@xxxxxxxxx> Signed-off-by: Hiroshi DOYU <Hiroshi.DOYU@xxxxxxxxx> --- drivers/dsp/bridge/pmgr/dbll.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/drivers/dsp/bridge/pmgr/dbll.c b/drivers/dsp/bridge/pmgr/dbll.c index 82430a3..c57f557 100644 --- a/drivers/dsp/bridge/pmgr/dbll.c +++ b/drivers/dsp/bridge/pmgr/dbll.c @@ -459,6 +459,8 @@ DSP_STATUS DBLL_getSect(struct DBLL_LibraryObj *lib, char *name, u32 *pAddr, (*(zlLib->pTarget->attrs.fseek))(zlLib->fp, zlLib->ulPos, SEEK_SET); } + } else { + status = DSP_EHANDLE; } if (DSP_SUCCEEDED(status)) { uByteSize = 1; @@ -834,8 +836,9 @@ DSP_STATUS DBLL_readSect(struct DBLL_LibraryObj *lib, char *name, (*(zlLib->pTarget->attrs.fseek))(zlLib->fp, zlLib->ulPos, SEEK_SET); } + } else { + status = DSP_EHANDLE; } - if (DSP_FAILED(status)) goto func_cont; -- 1.6.0.4 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html