xf86drm: check return value of asprintf()
authorSeung-Woo Kim <sw0312.kim@samsung.com>
Tue, 18 Jan 2022 02:34:41 +0000 (11:34 +0900)
committerHoegeun Kwon <hoegeun.kwon@samsung.com>
Wed, 7 Dec 2022 09:00:18 +0000 (18:00 +0900)
Check return value of asprintf() to fix build warning about not
checked return.

Note: fix build warning in upstream.

Change-Id: Iae8da56e7f743aa249869c58f6557148f799bd4c
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
xf86drm.c

index 0faa597..cda220b 100644 (file)
--- a/xf86drm.c
+++ b/xf86drm.c
@@ -356,9 +356,10 @@ drmGetFormatModifierNameFromNvidia(uint64_t modifier)
      * testing against TEGRA_TILE */
     if ((modifier & 0x10) == 0x10) {
         char *mod_nvidia;
-        asprintf(&mod_nvidia, "BLOCK_LINEAR_2D,HEIGHT=%"PRIu64",KIND=%"PRIu64","
+        if (asprintf(&mod_nvidia, "BLOCK_LINEAR_2D,HEIGHT=%"PRIu64",KIND=%"PRIu64","
                  "GEN=%"PRIu64",SECTOR=%"PRIu64",COMPRESSION=%"PRIu64"", height,
-                 kind, gen, sector, compression);
+                 kind, gen, sector, compression) < 0)
+               return NULL;
         return mod_nvidia;
     }
 
@@ -543,7 +544,9 @@ drmGetFormatModifierNameFromAmlogic(uint64_t modifier)
     else
         opts_str = "0";
 
-    asprintf(&mod_amlogic, "FBC,LAYOUT=%s,OPTIONS=%s", layout_str, opts_str);
+    if (asprintf(&mod_amlogic, "FBC,LAYOUT=%s,OPTIONS=%s", layout_str, opts_str)
+                   < 0)
+           return NULL;
     return mod_amlogic;
 }