>From: [email protected] >[mailto:[email protected]] On Behalf Of Gomez >Castellanos, Ivan > >The patch "DSPBRIDGE: Check pointer instead of using MEM_IS_VALID_HANDLE >macro" uncovered an issue when calling the assertion macros after the >pointer was freed, but not set to NULL. > >This issue was not showing before the above patch because the macro >MEM_IS_VALID_HANDLE checked for the pointer and signature, which was >not good because it dereferenced a pointer that was already freed. > >This patch removes DBC_ENSURE() whenever it checks for a previously >freed pointer, or fix the code to avoid wrong assertions printing. > >Signed-off-by: Ivan Gomez Castellanos <[email protected]> >--- > drivers/dsp/bridge/pmgr/chnl.c | 2 -- > drivers/dsp/bridge/pmgr/dev.c | 3 --- > drivers/dsp/bridge/rmgr/dbdcd.c | 5 ++--- > drivers/dsp/bridge/rmgr/disp.c | 2 -- > drivers/dsp/bridge/rmgr/drv.c | 2 +- > drivers/dsp/bridge/rmgr/mgr.c | 2 -- > drivers/dsp/bridge/rmgr/nldr.c | 1 - > drivers/dsp/bridge/rmgr/strm.c | 11 ++++------- > 8 files changed, 7 insertions(+), 21 deletions(-) >
Pushed to dspbridge. - omar -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
