summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAshutosh Dixit <ashutosh.dixit@intel.com>2022-07-01 20:05:34 -0700
committerAshutosh Dixit <ashutosh.dixit@intel.com>2022-07-06 14:11:21 -0700
commit1bd1a033e71790bcc60acccac32d4195cae40b6e (patch)
tree3a53cd5b98747d43432f05392575e1ce982723a1
parentf214bdcb75124f6e83921fea828195417dd4b346 (diff)
lib/igt_device_scan: Fix -Wdangling-pointer
[12/1181] Compiling C object lib/libigt_device_scan.a.p/igt_device_scan.c.o ../lib/igt_device_scan.c: In function ‘igt_device_add_attr’: ../lib/igt_device_scan.c:368:57: warning: dangling pointer ‘v’ to ‘linkto’ may be used [-Wdangling-pointer=] 368 | g_hash_table_insert(dev->attrs_ht, strdup(key), strdup(v)); | ^~~~~~~~~ ../lib/igt_device_scan.c:351:22: note: ‘linkto’ declared here 351 | char linkto[PATH_MAX]; | ^~~~~~ Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
-rw-r--r--lib/igt_device_scan.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/igt_device_scan.c b/lib/igt_device_scan.c
index a1cee7a4..b51275e4 100644
--- a/lib/igt_device_scan.c
+++ b/lib/igt_device_scan.c
@@ -337,6 +337,7 @@ static void igt_device_add_prop(struct igt_device *dev,
static void igt_device_add_attr(struct igt_device *dev,
const char *key, const char *value)
{
+ char linkto[PATH_MAX];
const char *v = value;
if (!key)
@@ -348,7 +349,6 @@ static void igt_device_add_attr(struct igt_device *dev,
if (!v) {
struct stat st;
char path[PATH_MAX];
- char linkto[PATH_MAX];
int len;
snprintf(path, sizeof(path), "%s/%s", dev->syspath, key);