user-util: move UID/GID related macros from macro.h to user-util.h

diff --git a/src/basic/fs-util.c b/src/basic/fs-util.c
index bd775b7b6febc0d996734c2f46fe294abef15b5a..edb007f98c757d6c13c42f7c76320e71facbe96c 100644 (file)
--- a/src/basic/fs-util.c
+++ b/src/basic/fs-util.c
@@ -27,6 +27,7 @@
 #include "path-util.h"
 #include "string-util.h"
 #include "strv.h"
+#include "user-util.h"
 #include "util.h"
 
 int unlink_noerrno(const char *path) {

diff --git a/src/basic/macro.h b/src/basic/macro.h
index f55d65e2f1545cb4ebe0e0901096dfe04fb4e1b5..eeb57ecb8ebacf10edbaab61968b62e55bce3297 100644 (file)
--- a/src/basic/macro.h
+++ b/src/basic/macro.h
@@ -295,14 +295,6 @@ static inline unsigned long ALIGN_POWER2(unsigned long u) {
 #define PTR_TO_SIZE(p) ((size_t) ((uintptr_t) (p)))
 #define SIZE_TO_PTR(u) ((void *) ((uintptr_t) (u)))
 
-/* The following macros add 1 when converting things, since UID 0 is a
- * valid UID, while the pointer NULL is special */
-#define PTR_TO_UID(p) ((uid_t) (((uintptr_t) (p))-1))
-#define UID_TO_PTR(u) ((void*) (((uintptr_t) (u))+1))
-
-#define PTR_TO_GID(p) ((gid_t) (((uintptr_t) (p))-1))
-#define GID_TO_PTR(u) ((void*) (((uintptr_t) (u))+1))
-
 #define PTR_TO_PID(p) ((pid_t) ((uintptr_t) p))
 #define PID_TO_PTR(p) ((void*) ((uintptr_t) p))
 
@@ -466,8 +458,6 @@ do {                                                                    \
 #endif
 #endif
 
-#define UID_INVALID ((uid_t) -1)
-#define GID_INVALID ((gid_t) -1)
 #define MODE_INVALID ((mode_t) -1)
 
 #define DEFINE_TRIVIAL_CLEANUP_FUNC(type, func)                 \

diff --git a/src/basic/mkdir.c b/src/basic/mkdir.c
index 9410b33503e6f190482a65d165f546b5b3183f1c..0214c4627ef1e0d16564b9da38b0b82af5ddf29c 100644 (file)
--- a/src/basic/mkdir.c
+++ b/src/basic/mkdir.c
@@ -26,6 +26,7 @@
 #include "mkdir.h"
 #include "path-util.h"
 #include "stat-util.h"
+#include "user-util.h"
 #include "util.h"
 
 int mkdir_safe_internal(const char *path, mode_t mode, uid_t uid, gid_t gid, mkdir_func_t _mkdir) {

diff --git a/src/basic/socket-util.c b/src/basic/socket-util.c
index 71a52a840ae3eefc73c44be6be8114cb400946c4..ef5b764e60cf0d7c5dceabdc6699569453ed7126 100644 (file)
--- a/src/basic/socket-util.c
+++ b/src/basic/socket-util.c
@@ -40,6 +40,7 @@
 #include "socket-util.h"
 #include "string-table.h"
 #include "string-util.h"
+#include "user-util.h"
 #include "util.h"
 
 int socket_address_parse(SocketAddress *a, const char *s) {

diff --git a/src/basic/user-util.h b/src/basic/user-util.h
index 4496f58ba3c1e880f28ed9ebe86e6a272dabf740..11ff6674cf5b7272989c9ce76b9223951e8c3691 100644 (file)
--- a/src/basic/user-util.h
+++ b/src/basic/user-util.h
@@ -54,3 +54,14 @@ int get_shell(char **_ret);
 int reset_uid_gid(void);
 
 int take_etc_passwd_lock(const char *root);
+
+#define UID_INVALID ((uid_t) -1)
+#define GID_INVALID ((gid_t) -1)
+
+/* The following macros add 1 when converting things, since UID 0 is a
+ * valid UID, while the pointer NULL is special */
+#define PTR_TO_UID(p) ((uid_t) (((uintptr_t) (p))-1))
+#define UID_TO_PTR(u) ((void*) (((uintptr_t) (u))+1))
+
+#define PTR_TO_GID(p) ((gid_t) (((uintptr_t) (p))-1))
+#define GID_TO_PTR(u) ((void*) (((uintptr_t) (u))+1))

diff --git a/src/bus-proxyd/proxy.c b/src/bus-proxyd/proxy.c
index ea2a01fdae33cb121c70da9e314b466eec802da8..6c251b52445d7d5a7b4df2f9a02f776df898dbdd 100644 (file)
--- a/src/bus-proxyd/proxy.c
+++ b/src/bus-proxyd/proxy.c
@@ -44,6 +44,7 @@
 #include "set.h"
 #include "strv.h"
 #include "synthesize.h"
+#include "user-util.h"
 #include "util.h"
 
 static int proxy_create_destination(Proxy *p, const char *destination, const char *local_sec, bool negotiate_fds) {

diff --git a/src/core/dbus-unit.c b/src/core/dbus-unit.c
index 62086a7fde39902a29f87b6ec57bff913fb2cefd..10cd80389bd179f5d6d93c8fb70c18861bef5117 100644 (file)
--- a/src/core/dbus-unit.c
+++ b/src/core/dbus-unit.c
@@ -31,6 +31,7 @@
 #include "special.h"
 #include "string-util.h"
 #include "strv.h"
+#include "user-util.h"
 
 static BUS_DEFINE_PROPERTY_GET_ENUM(property_get_load_state, unit_load_state, UnitLoadState);
 static BUS_DEFINE_PROPERTY_GET_ENUM(property_get_job_mode, job_mode, JobMode);

diff --git a/src/core/dbus.c b/src/core/dbus.c
index 37410a9870de8ba341ae84fab621cec02e3b89ee..bbab0aeddbe1ac5d57e63e1a246757bc5ac271f0 100644 (file)
--- a/src/core/dbus.c
+++ b/src/core/dbus.c
@@ -45,6 +45,7 @@
 #include "string-util.h"
 #include "strv.h"
 #include "strxcpyx.h"
+#include "user-util.h"
 
 #define CONNECTIONS_MAX 4096
 

diff --git a/src/core/mount-setup.c b/src/core/mount-setup.c
index 7486ec398580532f3cb5aabb2ddafc8d9ee6bfb7..1ab4ee5b1a8a1bd55abe4f4f7764560dda37c0b8 100644 (file)
--- a/src/core/mount-setup.c
+++ b/src/core/mount-setup.c
@@ -40,6 +40,7 @@
 #include "set.h"
 #include "smack-util.h"
 #include "strv.h"
+#include "user-util.h"
 #include "util.h"
 #include "virt.h"
 

diff --git a/src/core/namespace.c b/src/core/namespace.c
index 65cf140123676e55cb61548609b5e42d38480ccc..6c49ff32feb307460ca1266357e4c3117ee92016 100644 (file)
--- a/src/core/namespace.c
+++ b/src/core/namespace.c
@@ -42,6 +42,7 @@
 #include "string-util.h"
 #include "strv.h"
 #include "umask-util.h"
+#include "user-util.h"
 #include "util.h"
 
 typedef enum MountMode {

diff --git a/src/hostname/hostnamed.c b/src/hostname/hostnamed.c
index a42124288da4c1e037153e0e2ff9a8899d3c7113..5c63a025a33feb9bca8a7f679d08939ba351e40b 100644 (file)
--- a/src/hostname/hostnamed.c
+++ b/src/hostname/hostnamed.c
@@ -34,6 +34,7 @@
 #include "path-util.h"
 #include "selinux-util.h"
 #include "strv.h"
+#include "user-util.h"
 #include "util.h"
 #include "virt.h"
 

diff --git a/src/import/importd.c b/src/import/importd.c
index 601652231b02767f8474c2dc7f7c50086d57cf8f..65e235464030cbf8c85bdf412d87384faebf5707 100644 (file)
--- a/src/import/importd.c
+++ b/src/import/importd.c
@@ -40,6 +40,7 @@
 #include "string-table.h"
 #include "strv.h"
 #include "syslog-util.h"
+#include "user-util.h"
 #include "util.h"
 #include "web-util.h"
 

diff --git a/src/libsystemd/sd-bus/bus-control.c b/src/libsystemd/sd-bus/bus-control.c
index dc4ac88712cba4e71c1410ee70f120d38a6bbded..7c9dcf52837eb84540cf89e08bbaffbaf7ffaeec 100644 (file)
--- a/src/libsystemd/sd-bus/bus-control.c
+++ b/src/libsystemd/sd-bus/bus-control.c
@@ -29,13 +29,14 @@
 #include "sd-bus.h"
 
 #include "bus-bloom.h"
+#include "bus-control.h"
 #include "bus-internal.h"
 #include "bus-message.h"
 #include "bus-util.h"
 #include "capability-util.h"
 #include "string-util.h"
 #include "strv.h"
-#include "bus-control.h"
+#include "user-util.h"
 
 _public_ int sd_bus_get_unique_name(sd_bus *bus, const char **unique) {
         int r;

diff --git a/src/libsystemd/sd-bus/bus-creds.c b/src/libsystemd/sd-bus/bus-creds.c
index d291da8934bd45b783036c6839e432825cdcd7c4..7c99904c642278dcbe92c2ef7959ed90c5d2953f 100644 (file)
--- a/src/libsystemd/sd-bus/bus-creds.c
+++ b/src/libsystemd/sd-bus/bus-creds.c
@@ -38,6 +38,7 @@
 #include "string-util.h"
 #include "strv.h"
 #include "terminal-util.h"
+#include "user-util.h"
 #include "util.h"
 
 enum {

diff --git a/src/libsystemd/sd-bus/bus-kernel.c b/src/libsystemd/sd-bus/bus-kernel.c
index a455259007947dba4e66a97c969d316110dd86a8..a32b0ef2218533c1008fab2c6eac81bb2fc7dcd8 100644 (file)
--- a/src/libsystemd/sd-bus/bus-kernel.c
+++ b/src/libsystemd/sd-bus/bus-kernel.c
@@ -48,6 +48,7 @@
 #include "parse-util.h"
 #include "string-util.h"
 #include "strv.h"
+#include "user-util.h"
 #include "util.h"
 
 #define UNIQUE_NAME_MAX (3+DECIMAL_STR_MAX(uint64_t))

diff --git a/src/libsystemd/sd-device/device-private.c b/src/libsystemd/sd-device/device-private.c
index daed321bd84df1570aa1a4bdea1d4718840d0f1c..69288b833e00491e17f4a8e3ead3a064c6b14c46 100644 (file)
--- a/src/libsystemd/sd-device/device-private.c
+++ b/src/libsystemd/sd-device/device-private.c
@@ -41,6 +41,7 @@
 #include "string-util.h"
 #include "strv.h"
 #include "strxcpyx.h"
+#include "user-util.h"
 #include "util.h"
 
 int device_add_property(sd_device *device, const char *key, const char *value) {

diff --git a/src/locale/localed.c b/src/locale/localed.c
index 343399a62d31f695955e0e1d4c48bf958cf9b8d3..1f25d232d20121383ef9e11a3399920c823c3998 100644 (file)
--- a/src/locale/localed.c
+++ b/src/locale/localed.c
@@ -44,6 +44,7 @@
 #include "path-util.h"
 #include "selinux-util.h"
 #include "strv.h"
+#include "user-util.h"
 #include "util.h"
 
 enum {

diff --git a/src/login/logind-seat-dbus.c b/src/login/logind-seat-dbus.c
index 346e1d2cece27f970ee2d537c13a106a66a7f6b1..66626b5871dc90af14b6f1a443a16919cb24858d 100644 (file)
--- a/src/login/logind-seat-dbus.c
+++ b/src/login/logind-seat-dbus.c
@@ -22,13 +22,14 @@
 #include <errno.h>
 #include <string.h>
 
-#include "util.h"
-#include "bus-util.h"
-#include "strv.h"
 #include "bus-common-errors.h"
 #include "bus-label.h"
-#include "logind.h"
+#include "bus-util.h"
 #include "logind-seat.h"
+#include "logind.h"
+#include "strv.h"
+#include "user-util.h"
+#include "util.h"
 
 static int property_get_active_session(
                 sd_bus *bus,

diff --git a/src/login/logind-user.c b/src/login/logind-user.c
index 9692a867092fad17ef1b8bf6ff49a62e487962f6..1cf9be7441968650ddb4c69bb91cb4c34e407e9e 100644 (file)
--- a/src/login/logind-user.c
+++ b/src/login/logind-user.c
@@ -45,6 +45,7 @@
 #include "special.h"
 #include "string-table.h"
 #include "unit-name.h"
+#include "user-util.h"
 #include "util.h"
 
 User* user_new(Manager *m, uid_t uid, gid_t gid, const char *name) {

diff --git a/src/machine/image-dbus.c b/src/machine/image-dbus.c
index 2453a9ff04580d442daf9b99c6e07a883412d113..a287b9697a7d5c39f5af6a9d1c8b16ae7c58d1b9 100644 (file)
--- a/src/machine/image-dbus.c
+++ b/src/machine/image-dbus.c
@@ -20,10 +20,11 @@
 ***/
 
 #include "bus-label.h"
-#include "strv.h"
 #include "bus-util.h"
-#include "machine-image.h"
 #include "image-dbus.h"
+#include "machine-image.h"
+#include "strv.h"
+#include "user-util.h"
 
 static BUS_DEFINE_PROPERTY_GET_ENUM(property_get_type, image_type, ImageType);
 

diff --git a/src/machine/machine-dbus.c b/src/machine/machine-dbus.c
index 0003b84dd5b90c9f85d37ec83c3cf1d1bbdf89fc..1132d758237e0ba1a9ed7a481a73cc309a98ef1f 100644 (file)
--- a/src/machine/machine-dbus.c
+++ b/src/machine/machine-dbus.c
@@ -48,6 +48,7 @@
 #include "process-util.h"
 #include "strv.h"
 #include "terminal-util.h"
+#include "user-util.h"
 
 static int property_get_id(
                 sd_bus *bus,

diff --git a/src/nspawn/nspawn-mount.c b/src/nspawn/nspawn-mount.c
index 865407ec10934f7c6a9b2ddabc7582c2890a77ce..81003dff89ac5e571767dfecc65488f94a591003 100644 (file)
--- a/src/nspawn/nspawn-mount.c
+++ b/src/nspawn/nspawn-mount.c
@@ -36,6 +36,7 @@
 #include "stat-util.h"
 #include "string-util.h"
 #include "strv.h"
+#include "user-util.h"
 #include "util.h"
 
 CustomMount* custom_mount_add(CustomMount **l, unsigned *n, CustomMountType t) {

diff --git a/src/shared/base-filesystem.c b/src/shared/base-filesystem.c
index 9ccf26526f3dac73d99c0751cb5d3bcf61ed266d..ff91c6bdc50fb4a2c95e1259c88171241a206c4c 100644 (file)
--- a/src/shared/base-filesystem.c
+++ b/src/shared/base-filesystem.c
@@ -30,6 +30,7 @@
 #include "macro.h"
 #include "string-util.h"
 #include "umask-util.h"
+#include "user-util.h"
 #include "util.h"
 
 typedef struct BaseFilesystem {

diff --git a/src/shared/bus-util.c b/src/shared/bus-util.c
index c854ff43442df80d76382e1770dc962d73b05e9e..b253eb95eed3db423feb18a12ab178a07091b6c9 100644 (file)
--- a/src/shared/bus-util.c
+++ b/src/shared/bus-util.c
@@ -39,6 +39,7 @@
 #include "missing.h"
 #include "parse-util.h"
 #include "path-util.h"
+#include "proc-cmdline.h"
 #include "rlimit-util.h"
 #include "set.h"
 #include "signal-util.h"
@@ -46,9 +47,9 @@
 #include "strv.h"
 #include "syslog-util.h"
 #include "unit-name.h"
+#include "user-util.h"
 #include "utf8.h"
 #include "util.h"
-#include "proc-cmdline.h"
 
 static int name_owner_change_callback(sd_bus_message *m, void *userdata, sd_bus_error *ret_error) {
         sd_event *e = userdata;

diff --git a/src/shared/dev-setup.c b/src/shared/dev-setup.c
index 25ad918b85d16946b478a4e0b93df86a0ce0502e..0d321af9671083d8ef3a3e9a60c014e795c634fe 100644 (file)
--- a/src/shared/dev-setup.c
+++ b/src/shared/dev-setup.c
@@ -23,10 +23,11 @@
 #include <stdlib.h>
 #include <unistd.h>
 
-#include "util.h"
+#include "dev-setup.h"
 #include "label.h"
 #include "path-util.h"
-#include "dev-setup.h"
+#include "user-util.h"
+#include "util.h"
 
 int dev_setup(const char *prefix, uid_t uid, gid_t gid) {
         static const char symlinks[] =

diff --git a/src/shared/switch-root.c b/src/shared/switch-root.c
index 60d7043fda289b474ab037738c86b57cf84051cb..fc885f6cb8140f2a79e81f7b16354916215477d7 100644 (file)
--- a/src/shared/switch-root.c
+++ b/src/shared/switch-root.c
@@ -35,6 +35,7 @@
 #include "rm-rf.h"
 #include "string-util.h"
 #include "switch-root.h"
+#include "user-util.h"
 #include "util.h"
 
 int switch_root(const char *new_root, const char *oldroot, bool detach_oldroot,  unsigned long mountflags) {

diff --git a/src/test/test-cgroup-util.c b/src/test/test-cgroup-util.c
index 104f6327be581faa4f14a70336d5ae142742cb0d..e4f6d5168d8b1dc37aa61bf771a34d86c8022944 100644 (file)
--- a/src/test/test-cgroup-util.c
+++ b/src/test/test-cgroup-util.c
@@ -27,6 +27,7 @@
 #include "process-util.h"
 #include "string-util.h"
 #include "test-helper.h"
+#include "user-util.h"
 #include "util.h"
 
 static void check_p_d_u(const char *path, int code, const char *result) {

diff --git a/src/test/test-conf-files.c b/src/test/test-conf-files.c
index d00517e929cbaf1e1dc87a78201a34b0f3c15e94..f4b23355fc9b95e6bc25c7a0990844f646858de6 100644 (file)
--- a/src/test/test-conf-files.c
+++ b/src/test/test-conf-files.c
@@ -28,6 +28,7 @@
 #include "rm-rf.h"
 #include "string-util.h"
 #include "strv.h"
+#include "user-util.h"
 #include "util.h"
 
 static void setup_test_dir(char *tmp_dir, const char *files, ...) {

diff --git a/src/test/test-uid-range.c b/src/test/test-uid-range.c
index bc5baa2fcbe2fee1a570aa87b2439e497509a99f..9f69d71203dc29036a08b61ec36d43fd5e7b0482 100644 (file)
--- a/src/test/test-uid-range.c
+++ b/src/test/test-uid-range.c
@@ -21,8 +21,9 @@
 
 #include <stddef.h>
 
-#include "util.h"
 #include "uid-range.h"
+#include "user-util.h"
+#include "util.h"
 
 int main(int argc, char *argv[]) {
         _cleanup_free_ UidRange *p = NULL;

diff --git a/src/timedate/timedated.c b/src/timedate/timedated.c
index e97dc97ae12d2662699298804597ada71dcc714c..507a83b2a1cb3d3862effe3559e9cc6fb473daa8 100644 (file)
--- a/src/timedate/timedated.c
+++ b/src/timedate/timedated.c
@@ -38,6 +38,7 @@
 #include "path-util.h"
 #include "selinux-util.h"
 #include "strv.h"
+#include "user-util.h"
 #include "util.h"
 
 #define NULL_ADJTIME_UTC "0.0 0 0\n0\nUTC\n"

diff --git a/src/udev/udevd.c b/src/udev/udevd.c
index 2dcdef9918a99ac29a3c9a035138b16b6e1b7bd7..64346a782200353cd4e07059ff321d8dc767e7b3 100644 (file)
--- a/src/udev/udevd.c
+++ b/src/udev/udevd.c
@@ -63,6 +63,7 @@
 #include "terminal-util.h"
 #include "udev-util.h"
 #include "udev.h"
+#include "user-util.h"
 
 static bool arg_debug = false;
 static int arg_daemonize = false;