drm/i915/bios: fix format string of the VBT signature logging
authorJani Nikula <jani.nikula@intel.com>
Wed, 16 Dec 2015 13:04:19 +0000 (15:04 +0200)
committerJani Nikula <jani.nikula@intel.com>
Wed, 16 Dec 2015 16:01:32 +0000 (18:01 +0200)
Specify the maximum number of letters to print from the potentially
unterminated buffer, not the minimum. While at it, use sizeof instead of
a magic number.

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1450271061-32646-2-git-send-email-jani.nikula@intel.com
drivers/gpu/drm/i915/intel_bios.c

index dc3a0fb..4c9304e 100644 (file)
@@ -1326,8 +1326,8 @@ intel_bios_init(struct drm_device *dev)
 
        bdb = get_bdb_header(vbt);
 
-       DRM_DEBUG_KMS("VBT signature \"%20s\", BDB version %d\n",
-                     vbt->signature, bdb->version);
+       DRM_DEBUG_KMS("VBT signature \"%.*s\", BDB version %d\n",
+                     (int)sizeof(vbt->signature), vbt->signature, bdb->version);
 
        /* Grab useful general definitions */
        parse_general_features(dev_priv, bdb);