mesa: Don't leak shared state when context initialization fails
authorIan Romanick <ian.d.romanick@intel.com>
Wed, 10 Apr 2013 14:47:08 +0000 (07:47 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Fri, 12 Apr 2013 23:24:48 +0000 (16:24 -0700)
commit6e06550e4e0682c616fc0228beee419618421f2c
treea9c881c04d01c8e8cb094adcccbe454583f121ab
parentf730c210b89ab8eb2dd5f346a68d06bd05386a37
mesa: Don't leak shared state when context initialization fails

Back up at line 1017 (not shown in patch), we add a reference to the
shared state.  Several places after that may divert to the error
handler, but, as far as I can tell, nothing ever unreferences the shared
state.

Fixes issue identified by Klocwork analysis:

    Resource acquired to 'shared->TexMutex' at line 1012 may be lost
    here. Also there is one similar error on line 1087.

NOTE: This is a candidate for the stable branches.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
src/mesa/main/context.c