[Mesa-dev] [PATCH mesa v2] gallivm: ensure string is null-terminated instead of assert()ing

[Dylan Baker]

Quoting Eric Engestrom (2018-09-24 09:44:37)
> Signed-off-by: Eric Engestrom <eric.engestrom at intel.com>
> ---
> v2: Use the util_* wrapper to make sure MSVC stays happy (Dylan)
> ---
>  src/gallium/auxiliary/gallivm/lp_bld_printf.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/src/gallium/auxiliary/gallivm/lp_bld_printf.c b/src/gallium/auxiliary/gallivm/lp_bld_printf.c
> index 575ebdfdf65b7fd43b94..53c4d3cde41e4677abdb 100644
> --- a/src/gallium/auxiliary/gallivm/lp_bld_printf.c
> +++ b/src/gallium/auxiliary/gallivm/lp_bld_printf.c
> @@ -108,9 +108,7 @@ lp_build_print_value(struct gallivm_state *gallivm,
>        type_fmt[5] = '\0';
>     } else if (type_kind == LLVMIntegerTypeKind) {
>        if (LLVMGetIntTypeWidth(type_ref) == 64) {
> -         unsigned flen = strlen(PRId64);
> -         assert(flen <= 3);
> -         strncpy(type_fmt + 2, PRId64, flen);
> +         util_snprintf(type_fmt + 2, 3, "%s", PRId64);
>        } else if (LLVMGetIntTypeWidth(type_ref) == 8) {
>           type_fmt[2] = 'u';
>        } else {
> -- 
> Cheers,
>   Eric
> 

Reviewed-by: Dylan Baker <dylan at pnwbakers.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180924/3442caaf/attachment.sig>