[Mesa-dev] [PATCH 1/2] i965/disk_cache: Add extra unused character in renderer temp string

[Lionel Landwerlin]

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

On 22/07/18 05:45, Jordan Justen wrote:
> This extra character should not be used by snprintf, but we make it
> available to verify that we printed the exact number we wanted, and
> didn't overflow.
>
> Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
> ---
>   src/mesa/drivers/dri/i965/brw_disk_cache.c | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_disk_cache.c b/src/mesa/drivers/dri/i965/brw_disk_cache.c
> index c478753d4ad..a678c355b9d 100644
> --- a/src/mesa/drivers/dri/i965/brw_disk_cache.c
> +++ b/src/mesa/drivers/dri/i965/brw_disk_cache.c
> @@ -377,10 +377,11 @@ void
>   brw_disk_cache_init(struct intel_screen *screen)
>   {
>   #ifdef ENABLE_SHADER_CACHE
> -   char renderer[10];
> +   /* array length: print length + null char + 1 extra to verify it is unused */
> +   char renderer[11];
>      MAYBE_UNUSED int len = snprintf(renderer, sizeof(renderer), "i965_%04x",
>                                      screen->deviceID);
> -   assert(len == sizeof(renderer) - 1);
> +   assert(len == sizeof(renderer) - 2);
>   
>      const struct build_id_note *note =
>         build_id_find_nhdr_for_addr(brw_disk_cache_init);