]> git.ipfire.org Git - thirdparty/lxc.git/commit
criu: fix error message 3996/head
authorTycho Andersen <tycho@tycho.pizza>
Thu, 14 Oct 2021 16:40:08 +0000 (10:40 -0600)
committerTycho Andersen <tycho@tycho.pizza>
Thu, 14 Oct 2021 16:41:36 +0000 (10:41 -0600)
commitbdb8aeda721f9262695001a7368bb991527b4b8a
tree085a871dd23fc7ceb2f1e25d235010e959e10dc8
parentb2668c872f271366332df85997962330739f3afb
criu: fix error message

as of 59d8a539d106 ("criu: massage exec_criu()") I see:

In file included from criu.c:22:
criu.c: In function 'exec_criu':
log.h:376:2: error: '%s' directive argument is null [-Werror=format-overflow=]
  376 |  LXC_ERROR(&locinfo, format, ##__VA_ARGS__);   \
      |  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
log.h:457:3: note: in expansion of macro 'ERROR'
  457 |   ERROR("%s - " format, ptr, ##__VA_ARGS__); \
      |   ^~~~~
log.h:491:3: note: in expansion of macro 'SYSERROR'
  491 |   SYSERROR(format, ##__VA_ARGS__);              \
      |   ^~~~~~~~
criu.c:325:11: note: in expansion of macro 'log_error_errno'
  325 |    return log_error_errno(-ENOMEM, ENOMEM, "Failed to remove extraneous slashes from \"%s\"", tmp);
      |           ^~~~~~~~~~~~~~~

it looks like we should be logging the string that failed, vs. tmp here.

(my log was taken from stable-4.0, but the same issue exists on master it
seems.)

Signed-off-by: Tycho Andersen <tycho@tycho.pizza>
src/lxc/criu.c