diff options
author | Matt Turner <[email protected]> | 2015-03-03 16:02:40 -0800 |
---|---|---|
committer | Matt Turner <[email protected]> | 2015-03-04 11:15:09 -0800 |
commit | f5e2aa1324dd6a9666bb21834097d2fbc3cb99b6 (patch) | |
tree | cb8d4631b873541fc751aca436fe9379592c37fd /src/gallium/drivers/r300 | |
parent | 67e0a4f6e8a6c6d03cf3f9d15797bc14c373711e (diff) |
r300g: Use PATH_MAX instead of limiting ourselves to 100 chars.
When built with Gentoo's package manager, the Mesa source directory
exists seven directories deep. The path to the .test file is too long
and is silently truncated, leading to a crash. Just use PATH_MAX.
Cc: 10.4, 10.5 <[email protected]>
Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=540970
Reviewed-by: Tom Stellard <[email protected]>
Diffstat (limited to 'src/gallium/drivers/r300')
-rw-r--r-- | src/gallium/drivers/r300/compiler/tests/rc_test_helpers.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/gallium/drivers/r300/compiler/tests/rc_test_helpers.c b/src/gallium/drivers/r300/compiler/tests/rc_test_helpers.c index 7c9d1779df5..422bdb04405 100644 --- a/src/gallium/drivers/r300/compiler/tests/rc_test_helpers.c +++ b/src/gallium/drivers/r300/compiler/tests/rc_test_helpers.c @@ -28,6 +28,7 @@ */ #include <errno.h> +#include <limits.h> #include <regex.h> #include <stdlib.h> #include <stdio.h> @@ -528,7 +529,6 @@ void init_compiler( } #define MAX_LINE_LENGTH 100 -#define MAX_PATH_LENGTH 100 unsigned load_program( struct radeon_compiler *c, @@ -536,14 +536,14 @@ unsigned load_program( const char *filename) { char line[MAX_LINE_LENGTH]; - char path[MAX_PATH_LENGTH]; + char path[PATH_MAX]; FILE *file; unsigned *count; char **string_store; unsigned i = 0; memset(line, 0, sizeof(line)); - snprintf(path, MAX_PATH_LENGTH, TEST_PATH "/%s", filename); + snprintf(path, PATH_MAX, TEST_PATH "/%s", filename); file = fopen(path, "r"); if (!file) { return 0; |