util: add some defensive coding in u_upload_alloc()
authorBrian Paul <brianp@vmware.com>
Mon, 11 Feb 2013 19:27:29 +0000 (12:27 -0700)
committerAndreas Boll <andreas.boll.dev@gmail.com>
Wed, 13 Feb 2013 20:47:57 +0000 (21:47 +0100)
commita1cb12c1fbec30c23d50209db2fbf2b563ddcbe0
treee698f8f8f345d37dc90e71b4798cfdb49b5372ad
parentd32aff91c1c42b3f8bfb6c4109a8ea79c1f46d8d
util: add some defensive coding in u_upload_alloc()

Some callers of this function were checking the 'ptr' result to see if
the function failed.  But the correct way is to check the regular
return value for PIPE_ERROR_x.  Now we initialize all the returned
values at the top of the function in case we do hit an error (like OOM).

Callers are more likely to detect OOM conditions now.  But there
are some callers which don't do any error checking...

Note: This is a candidate for the 9.0 branch.

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
(cherry picked from commit 68a097596efdf314ec60487a1761cac789ae2367)

Conflicts:

src/gallium/auxiliary/util/u_upload_mgr.c
src/gallium/auxiliary/util/u_upload_mgr.c