All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] tools: Fix running intel_dp_compliance if it is found in the search path($PATH env variable)
@ 2021-04-08  0:21 Khaled Almahallawy
  2021-04-08  0:50 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (5 more replies)
  0 siblings, 6 replies; 10+ messages in thread
From: Khaled Almahallawy @ 2021-04-08  0:21 UTC (permalink / raw)
  To: igt-dev; +Cc: shawn.c.lee, petri.latvala, Khaled Almahallawy

When intel_dp_compliance binary location is part of PATH environment variable, running it from the shell without relative or full path, cause the following assertion failure because chdir:

$:~# intel_dp_compliance
(intel_dp_compliance:3070) CRITICAL: Test assertion failure function enter_exec_path, file ../tools/igt_compliance_utils.c:56:
(intel_dp_compliance:3070) CRITICAL: Failed assertion: ret == 0
(intel_dp_compliance:3070) CRITICAL: Last errno: 2, No such file or directory
(intel_dp_compliance:3070) CRITICAL: error: -1 != 0
Stack trace:
  #0 ../lib/igt_core.c:1726 __igt_fail_assert()
  #1 [enter_exec_path+0x8b]
  #2 ../tools/intel_dp_compliance.c:843 main()
  #3 ../csu/libc-start.c:342 __libc_start_main()
  #4 [_start+0x2e]
Test (null) failed.
**** DEBUG ****
(intel_dp_compliance:3070) CRITICAL: Test assertion failure function enter_exec_path, file ../tools/igt_compliance_utils.c:56:
(intel_dp_compliance:3070) CRITICAL: Failed assertion: ret == 0
(intel_dp_compliance:3070) CRITICAL: Last errno: 2, No such file or directory
(intel_dp_compliance:3070) CRITICAL: error: -1 != 0
(intel_dp_compliance:3070) igt_core-INFO: Stack trace:
(intel_dp_compliance:3070) igt_core-INFO:   #0 ../lib/igt_core.c:1726 __igt_fail_assert()
(intel_dp_compliance:3070) igt_core-INFO:   #1 [enter_exec_path+0x8b]
(intel_dp_compliance:3070) igt_core-INFO:   #2 ../tools/intel_dp_compliance.c:843 main()
(intel_dp_compliance:3070) igt_core-INFO:   #3 ../csu/libc-start.c:342 __libc_start_main()
(intel_dp_compliance:3070) igt_core-INFO:   #4 [_start+0x2e]
****  END  ****
FAIL (-1.000s)

This patch fixes this issue.

Signed-off-by: Khaled Almahallawy <khaled.almahallawy@intel.com>
---
 tools/igt_compliance_utils.c | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/tools/igt_compliance_utils.c b/tools/igt_compliance_utils.c
index d6a2e4e529d9..bcc707ae83c1 100644
--- a/tools/igt_compliance_utils.c
+++ b/tools/igt_compliance_utils.c
@@ -39,22 +39,20 @@ struct termios saved_tio;
 
 void enter_exec_path(char **argv)
 {
-	char *exec_path = NULL;
+	char exec_path[PATH_MAX];
 	char *pos = NULL;
-	short len_path = 0;
 	int ret;
 
-	len_path = strlen(argv[0]);
-	exec_path = (char *) malloc(len_path);
+	memset(exec_path, 0, sizeof(exec_path));
+	ret = readlink("/proc/self/exe", exec_path, PATH_MAX);
+	igt_assert(ret != -1);
 
-	memcpy(exec_path, argv[0], len_path);
 	pos = strrchr(exec_path, '/');
-	if (pos != NULL)
-		*(pos+1) = '\0';
+	if(pos != NULL)
+		*pos = '\0';
 
 	ret = chdir(exec_path);
 	igt_assert_eq(ret, 0);
-	free(exec_path);
 }
 
 static void restore_termio_mode(int sig)
-- 
2.25.1

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2021-04-09  8:02 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-08  0:21 [igt-dev] [PATCH i-g-t] tools: Fix running intel_dp_compliance if it is found in the search path($PATH env variable) Khaled Almahallawy
2021-04-08  0:50 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2021-04-08  3:23 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2021-04-08  9:10 ` [igt-dev] [PATCH i-g-t] " Petri Latvala
2021-04-08 21:51 ` [igt-dev] [PATCH i-g-t v2] " Khaled Almahallawy
2021-04-08 22:26   ` Navare, Manasi D
2021-04-08 23:26     ` Almahallawy, Khaled
2021-04-09  8:03       ` Petri Latvala
2021-04-08 23:38 ` [igt-dev] ✓ Fi.CI.BAT: success for tools: Fix running intel_dp_compliance if it is found in the search path($PATH env variable) (rev3) Patchwork
2021-04-09  2:37 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.