mesa: avoid NULL ptr in prog parameter name
authorGregory Hainaut <gregory.hainaut@gmail.com>
Tue, 11 Apr 2017 20:29:17 +0000 (22:29 +0200)
committerTimothy Arceri <tarceri@itsqueeze.com>
Wed, 12 Apr 2017 04:30:28 +0000 (14:30 +1000)
commit03d1de387ef35c69fb11781b31824f6f2f310006
treea9bff96bdbd759f90a3c7f711a357f4b40858589
parent754b961f384d567b148366fc96f29811e09839c5
mesa: avoid NULL ptr in prog parameter name

Context: _mesa_add_parameter is sometimes[0] called with a
NULL name as a mean of an unnamed parameter.

Allowing NULL pointer as a name means that it must be NULL checked
each access. So far it isn't always[1] true.

Parameter name is only used for debug purpose (printf) and
to lookup the index/location of the program by the application.

Conclusion, there is no valid reason to use a NULL pointer instead of
an empty string. So it was decided to use an empty string which avoid all
issues related to NULL pointer

[0]: texture gather offsets glsl opcode and st_init_atifs_prog
[1]: at least shader cache, st_nir_lookup_parameter_index and some printfs

Issue found by piglit 'texturegatheroffsets' tests on Nouveau

v4: new patch based on Nicolai/Timothy/ilia discussion
Signed-off-by: Gregory Hainaut <gregory.hainaut@gmail.com>
Reviewed-by: Timothy Arceri <tarceri@itsqueeze.com>
src/mesa/program/prog_parameter.c