diff options
| author | Ludovic Courtès <ludo@gnu.org> | 2015-02-24 23:05:45 +0100 |
|---|---|---|
| committer | Ludovic Courtès <ludo@gnu.org> | 2015-02-24 23:43:03 +0100 |
| commit | 2178ed66f7ae8f793078c7e392af4bf1fb472968 (patch) | |
| tree | 91589714052eb7d3fb36ca61f538ca72a578143c /m4 | |
| parent | 43da8f018d5835b62e8f5f1f4e2cc701f828a3db (diff) | |
build: Reject or warn against file name length limit overruns.
* m4/guix.m4 (GUIX_TEST_ROOT_DIRECTORY, LINUX_HASH_BANG_LIMIT,
SOCKET_FILE_NAME_LIMIT, GUIX_SOCKET_FILE_NAME_LENGTH,
GUIX_TEST_SOCKET_FILE_NAME_LENGTH, GUIX_HASH_BANG_LENGTH,
GUIX_CHECK_FILE_NAME_LIMITS): New macros.
* configure.ac: Use 'GUIX_CHECK_FILE_NAME_LIMITS'.
* config-daemon.ac: Use 'GUIX_TEST_ROOT_DIRECTORY'.
* test-env.in: Check socket name length and emit warning if it exceeds
107.
Diffstat (limited to 'm4')
| -rw-r--r-- | m4/guix.m4 | 58 |
1 files changed, 57 insertions, 1 deletions
diff --git a/m4/guix.m4 b/m4/guix.m4 index d4415598b04..445ce857dd6 100644 --- a/m4/guix.m4 +++ b/m4/guix.m4 | |||
| @@ -1,5 +1,5 @@ | |||
| 1 | dnl GNU Guix --- Functional package management for GNU | 1 | dnl GNU Guix --- Functional package management for GNU |
| 2 | dnl Copyright © 2012, 2013, 2014 Ludovic Courtès <ludo@gnu.org> | 2 | dnl Copyright © 2012, 2013, 2014, 2015 Ludovic Courtès <ludo@gnu.org> |
| 3 | dnl Copyright © 2014 Mark H Weaver <mhw@netris.org> | 3 | dnl Copyright © 2014 Mark H Weaver <mhw@netris.org> |
| 4 | dnl | 4 | dnl |
| 5 | dnl This file is part of GNU Guix. | 5 | dnl This file is part of GNU Guix. |
| @@ -162,3 +162,59 @@ AC_DEFUN([GUIX_CHECK_UNBUFFERED_CBIP], [ | |||
| 162 | ac_cv_guix_cbips_support_setvbuf=no | 162 | ac_cv_guix_cbips_support_setvbuf=no |
| 163 | fi]) | 163 | fi]) |
| 164 | ]) | 164 | ]) |
| 165 | |||
| 166 | dnl GUIX_TEST_ROOT_DIRECTORY | ||
| 167 | AC_DEFUN([GUIX_TEST_ROOT_DIRECTORY], [ | ||
| 168 | AC_CACHE_CHECK([for unit test root directory], | ||
| 169 | [ac_cv_guix_test_root], | ||
| 170 | [ac_cv_guix_test_root="`pwd`/test-tmp"]) | ||
| 171 | ]) | ||
| 172 | |||
| 173 | dnl 'BINPRM_BUF_SIZE' constant in Linux. The Hurd has a limit | ||
| 174 | dnl of about a page (see exec/hashexec.c.) | ||
| 175 | m4_define([LINUX_HASH_BANG_LIMIT], 128) | ||
| 176 | |||
| 177 | dnl Hardcoded 'sun_path' length in <sys/un.h>. | ||
| 178 | m4_define([SOCKET_FILE_NAME_LIMIT], 108) | ||
| 179 | |||
| 180 | dnl GUIX_SOCKET_FILE_NAME_LENGTH | ||
| 181 | AC_DEFUN([GUIX_SOCKET_FILE_NAME_LENGTH], [ | ||
| 182 | AC_CACHE_CHECK([the length of the installed socket file name], | ||
| 183 | [ac_cv_guix_socket_file_name_length], | ||
| 184 | [ac_cv_guix_socket_file_name_length="`echo -n "$guix_localstatedir/guix/daemon-socket/socket" | wc -c`"]) | ||
| 185 | ]) | ||
| 186 | |||
| 187 | dnl GUIX_TEST_SOCKET_FILE_NAME_LENGTH | ||
| 188 | AC_DEFUN([GUIX_TEST_SOCKET_FILE_NAME_LENGTH], [ | ||
| 189 | AC_REQUIRE([GUIX_TEST_ROOT_DIRECTORY]) | ||
| 190 | AC_CACHE_CHECK([the length of the socket file name used in tests], | ||
| 191 | [ac_cv_guix_test_socket_file_name_length], | ||
| 192 | [ac_cv_guix_test_socket_file_name_length="`echo -n "$ac_cv_guix_test_root/var/123456/daemon-socket/socket" | wc -c`"]) | ||
| 193 | ]) | ||
| 194 | |||
| 195 | dnl GUIX_HASH_BANG_LENGTH | ||
| 196 | AC_DEFUN([GUIX_HASH_BANG_LENGTH], [ | ||
| 197 | AC_CACHE_CHECK([the length of a typical hash bang line], | ||
| 198 | [ac_cv_guix_hash_bang_length], | ||
| 199 | [ac_cv_guix_hash_bang_length=`echo -n "$storedir/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-bootstrap-binaries-0/bin/bash" | wc -c`]) | ||
| 200 | ]) | ||
| 201 | |||
| 202 | dnl GUIX_CHECK_FILE_NAME_LIMITS | ||
| 203 | dnl | ||
| 204 | dnl GNU/Linux has a couple of silly limits that we can easily run into. | ||
| 205 | dnl Make sure everything is fine with the current settings. | ||
| 206 | AC_DEFUN([GUIX_CHECK_FILE_NAME_LIMITS], [ | ||
| 207 | AC_REQUIRE([GUIX_SOCKET_FILE_NAME_LENGTH]) | ||
| 208 | AC_REQUIRE([GUIX_TEST_SOCKET_FILE_NAME_LENGTH]) | ||
| 209 | AC_REQUIRE([GUIX_HASH_BANG_LENGTH]) | ||
| 210 | |||
| 211 | if test "$ac_cv_guix_socket_file_name_length" -ge ]SOCKET_FILE_NAME_LIMIT[; then | ||
| 212 | AC_MSG_ERROR([socket file name would exceed the maxium allowed length]) | ||
| 213 | fi | ||
| 214 | if test "$ac_cv_guix_test_socket_file_name_length" -ge ]SOCKET_FILE_NAME_LIMIT[; then | ||
| 215 | AC_MSG_WARN([socket file name limit may be exceeded when running tests]) | ||
| 216 | fi | ||
| 217 | if test "$ac_cv_guix_hash_bang_length" -ge ]LINUX_HASH_BANG_LIMIT[; then | ||
| 218 | AC_MSG_ERROR([store directory '$storedir' would lead to overly long hash-bang lines]) | ||
| 219 | fi | ||
| 220 | ]) | ||
