diff options
author | Peter A. Bigot <pab@pabigot.com> | 2014-10-12 16:58:52 -0500 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-11-25 13:03:29 +0000 |
commit | 835f5c84109693a807229f6ab9d75c850f45eff1 (patch) | |
tree | 5519b26794602246cbd54ab91e50b1ea76eb3ed9 /meta/recipes-devtools/pseudo | |
parent | d2b8576bfadd70eec3ef764c33e761eb22830536 (diff) | |
download | poky-835f5c84109693a807229f6ab9d75c850f45eff1.tar.gz |
pseudo: support multiple search directories in PSEUDO_PASSWD
This makes it possible to use --without-passwd-fallback when building
images where the preferred passwd files are not available until after
installation has begun.
(From OE-Core rev: 15b3b796d6e06fb7a7867d132b234d783e733531)
Signed-off-by: Peter A. Bigot <pab@pabigot.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-devtools/pseudo')
3 files changed, 188 insertions, 0 deletions
diff --git a/meta/recipes-devtools/pseudo/pseudo-1.6.2/0002-pseudo_util-modify-interface-to-pseudo_etc_file.patch b/meta/recipes-devtools/pseudo/pseudo-1.6.2/0002-pseudo_util-modify-interface-to-pseudo_etc_file.patch new file mode 100644 index 0000000000..c7006ef6df --- /dev/null +++ b/meta/recipes-devtools/pseudo/pseudo-1.6.2/0002-pseudo_util-modify-interface-to-pseudo_etc_file.patch | |||
@@ -0,0 +1,70 @@ | |||
1 | From f05def2bbd5507084672bc9072ffe0e5101e9b47 Mon Sep 17 00:00:00 2001 | ||
2 | From: "Peter A. Bigot" <pab@pabigot.com> | ||
3 | Date: Sun, 12 Oct 2014 11:35:57 -0500 | ||
4 | Subject: [PATCH 2/3] pseudo_util: modify interface to pseudo_etc_file | ||
5 | |||
6 | * Make the search directory pointers const: there is no reason why this | ||
7 | function should be allowed to mutate the directories. | ||
8 | |||
9 | * Change the search directory argument from an array of pointers to a | ||
10 | pointer-to-pointers to prepare for an upcoming enhancement. | ||
11 | |||
12 | Upstream-Status: Pending | ||
13 | Signed-off-by: Peter A. Bigot <pab@pabigot.com> | ||
14 | --- | ||
15 | pseudo.h | 2 +- | ||
16 | pseudo_client.c | 2 +- | ||
17 | pseudo_util.c | 4 ++-- | ||
18 | 3 files changed, 4 insertions(+), 4 deletions(-) | ||
19 | |||
20 | diff --git a/pseudo.h b/pseudo.h | ||
21 | index 92020e4..05813c1 100644 | ||
22 | --- a/pseudo.h | ||
23 | +++ b/pseudo.h | ||
24 | @@ -86,7 +86,7 @@ extern int pseudo_logfile(char *defname); | ||
25 | extern ssize_t pseudo_sys_path_max(void); | ||
26 | extern ssize_t pseudo_path_max(void); | ||
27 | #define PSEUDO_PWD_MAX 4096 | ||
28 | -extern int pseudo_etc_file(const char *filename, char *realname, int flags, char *path[], int dircount); | ||
29 | +extern int pseudo_etc_file(const char *filename, char *realname, int flags, const char **search_dirs, int dircount); | ||
30 | extern void pseudo_stat32_from64(struct stat *, const struct stat64 *); | ||
31 | extern void pseudo_stat64_from32(struct stat64 *, const struct stat *); | ||
32 | |||
33 | diff --git a/pseudo_client.c b/pseudo_client.c | ||
34 | index 442dd19..7a4d7fa 100644 | ||
35 | --- a/pseudo_client.c | ||
36 | +++ b/pseudo_client.c | ||
37 | @@ -93,7 +93,7 @@ gid_t pseudo_egid; | ||
38 | gid_t pseudo_sgid; | ||
39 | gid_t pseudo_fgid; | ||
40 | |||
41 | -#define PSEUDO_ETC_FILE(filename, realname, flags) pseudo_etc_file(filename, realname, flags, (char *[]) { pseudo_chroot, pseudo_passwd, PSEUDO_PASSWD_FALLBACK }, PSEUDO_PASSWD_FALLBACK ? 3 : 2) | ||
42 | +#define PSEUDO_ETC_FILE(filename, realname, flags) pseudo_etc_file(filename, realname, flags, (const char *[]) { pseudo_chroot, pseudo_passwd, PSEUDO_PASSWD_FALLBACK }, PSEUDO_PASSWD_FALLBACK ? 3 : 2) | ||
43 | |||
44 | /* helper function to make a directory, just like mkdir -p. | ||
45 | * Can't use system() because the child shell would end up trying | ||
46 | diff --git a/pseudo_util.c b/pseudo_util.c | ||
47 | index e4e1fc8..647d3ad 100644 | ||
48 | --- a/pseudo_util.c | ||
49 | +++ b/pseudo_util.c | ||
50 | @@ -1264,7 +1264,7 @@ FILE *pseudo_host_etc_group_file = &pseudo_fake_group_file; | ||
51 | #endif | ||
52 | |||
53 | int | ||
54 | -pseudo_etc_file(const char *file, char *realname, int flags, char *search_dirs[], int dircount) { | ||
55 | +pseudo_etc_file(const char *file, char *realname, int flags, const char **search_dirs, int dircount) { | ||
56 | char filename[pseudo_path_max()]; | ||
57 | int rc = -1; | ||
58 | |||
59 | @@ -1280,7 +1280,7 @@ pseudo_etc_file(const char *file, char *realname, int flags, char *search_dirs[] | ||
60 | return -1; | ||
61 | } | ||
62 | for (i = 0; i < dircount; ++i) { | ||
63 | - char *s = search_dirs[i]; | ||
64 | + const char *s = search_dirs[i]; | ||
65 | if (!s) | ||
66 | continue; | ||
67 | #if PSEUDO_PORT_DARWIN | ||
68 | -- | ||
69 | 1.8.5.5 | ||
70 | |||
diff --git a/meta/recipes-devtools/pseudo/pseudo-1.6.2/0003-pseudo_client.c-support-multiple-directories-in-PSEU.patch b/meta/recipes-devtools/pseudo/pseudo-1.6.2/0003-pseudo_client.c-support-multiple-directories-in-PSEU.patch new file mode 100644 index 0000000000..e6c6284a24 --- /dev/null +++ b/meta/recipes-devtools/pseudo/pseudo-1.6.2/0003-pseudo_client.c-support-multiple-directories-in-PSEU.patch | |||
@@ -0,0 +1,116 @@ | |||
1 | From 09f04dc36f21c179235109b3dcddce9dda9a8ba8 Mon Sep 17 00:00:00 2001 | ||
2 | From: "Peter A. Bigot" <pab@pabigot.com> | ||
3 | Date: Sun, 12 Oct 2014 12:17:48 -0500 | ||
4 | Subject: [PATCH 3/3] pseudo_client.c: support multiple directories in | ||
5 | PSEUDO_PASSWD | ||
6 | |||
7 | For OpenEmbedded it is highly unlikely that using the build host passwd | ||
8 | file is the right approach. Most packages can be built with a pseudo | ||
9 | that was configured --without-passwd-fallback, since | ||
10 | PSEUDO_PASSWD=${STAGING_DIR_TARGET} suffices. | ||
11 | |||
12 | This fails when building images, because image.bbclass (correctly) | ||
13 | overrides to PSEUDO_PASSWD=${IMAGE_ROOTFS}. However, the rootfs | ||
14 | /etc/passwd is not created until the post-install phase of base-passwd, | ||
15 | which is long after a passwd file is required. For example, the smart | ||
16 | RPM interface wants to look up uid 0 right away. The right solution | ||
17 | here is to look first in ${IMAGE_ROOTFS}, then fallback to | ||
18 | a location holding immutable files with the minimum user/group settings | ||
19 | necessary to successfully get base-passwd onto the target. | ||
20 | |||
21 | Rather than rework pseudo to change PSEUDO_PASSWD_FALLBACK to be a | ||
22 | run-time rather than compile-time specification, rework the handling of | ||
23 | PSEUDO_PASSWD so that it is a colon-separated list of directories that | ||
24 | are processed in order. | ||
25 | |||
26 | Upstream-Status: Pending | ||
27 | Signed-off-by: Peter A. Bigot <pab@pabigot.com> | ||
28 | --- | ||
29 | pseudo_client.c | 50 +++++++++++++++++++++++++++++++++++++++++++++++++- | ||
30 | 1 file changed, 49 insertions(+), 1 deletion(-) | ||
31 | |||
32 | diff --git a/pseudo_client.c b/pseudo_client.c | ||
33 | index 7a4d7fa..b52b86a 100644 | ||
34 | --- a/pseudo_client.c | ||
35 | +++ b/pseudo_client.c | ||
36 | @@ -75,6 +75,8 @@ int pseudo_umask = 022; | ||
37 | |||
38 | static char **fd_paths = NULL; | ||
39 | static int nfds = 0; | ||
40 | +static const char **passwd_paths = NULL; | ||
41 | +static int npasswd_paths = 0; | ||
42 | static int messages = 0; | ||
43 | static struct timeval message_time = { .tv_sec = 0 }; | ||
44 | static int pseudo_inited = 0; | ||
45 | @@ -93,7 +95,7 @@ gid_t pseudo_egid; | ||
46 | gid_t pseudo_sgid; | ||
47 | gid_t pseudo_fgid; | ||
48 | |||
49 | -#define PSEUDO_ETC_FILE(filename, realname, flags) pseudo_etc_file(filename, realname, flags, (const char *[]) { pseudo_chroot, pseudo_passwd, PSEUDO_PASSWD_FALLBACK }, PSEUDO_PASSWD_FALLBACK ? 3 : 2) | ||
50 | +#define PSEUDO_ETC_FILE(filename, realname, flags) pseudo_etc_file(filename, realname, flags, passwd_paths, npasswd_paths) | ||
51 | |||
52 | /* helper function to make a directory, just like mkdir -p. | ||
53 | * Can't use system() because the child shell would end up trying | ||
54 | @@ -117,6 +119,42 @@ mkdir_p(char *path) { | ||
55 | (void) mkdir(path, 0755); | ||
56 | } | ||
57 | |||
58 | +static int | ||
59 | +build_passwd_paths(const char **paths) | ||
60 | +{ | ||
61 | + int np = 0; | ||
62 | + | ||
63 | + if (pseudo_chroot) { | ||
64 | + if (paths) { | ||
65 | + paths[np] = pseudo_chroot; | ||
66 | + } | ||
67 | + ++np; | ||
68 | + } | ||
69 | + if (pseudo_passwd) { | ||
70 | + const char *cp = pseudo_passwd; | ||
71 | + const char *next = strchr(cp, ':'); | ||
72 | + while (next) { | ||
73 | + if (paths) { | ||
74 | + paths[np] = strndup(cp, next-cp); | ||
75 | + } | ||
76 | + ++np; | ||
77 | + cp = next+1; | ||
78 | + next = strchr(cp, ':'); | ||
79 | + } | ||
80 | + if (paths) { | ||
81 | + paths[np] = strdup(cp); | ||
82 | + } | ||
83 | + ++np; | ||
84 | + } | ||
85 | + if (PSEUDO_PASSWD_FALLBACK) { | ||
86 | + if (paths) { | ||
87 | + paths[np] = PSEUDO_PASSWD_FALLBACK; | ||
88 | + } | ||
89 | + ++np; | ||
90 | + } | ||
91 | + return np; | ||
92 | +} | ||
93 | + | ||
94 | void | ||
95 | pseudo_init_client(void) { | ||
96 | char *env; | ||
97 | @@ -329,6 +367,16 @@ pseudo_init_client(void) { | ||
98 | } | ||
99 | free(env); | ||
100 | |||
101 | + npasswd_paths = build_passwd_paths(NULL); | ||
102 | + if (npasswd_paths) { | ||
103 | + passwd_paths = malloc(npasswd_paths * sizeof(*passwd_paths)); | ||
104 | + if (!passwd_paths) { | ||
105 | + pseudo_diag("couldn't allocate space for passwd paths.\n"); | ||
106 | + exit(1); | ||
107 | + } | ||
108 | + build_passwd_paths(passwd_paths); | ||
109 | + } | ||
110 | + | ||
111 | pseudo_inited = 1; | ||
112 | } | ||
113 | if (!pseudo_disabled) | ||
114 | -- | ||
115 | 1.8.5.5 | ||
116 | |||
diff --git a/meta/recipes-devtools/pseudo/pseudo_1.6.2.bb b/meta/recipes-devtools/pseudo/pseudo_1.6.2.bb index df8ce832c1..78eeedf0df 100644 --- a/meta/recipes-devtools/pseudo/pseudo_1.6.2.bb +++ b/meta/recipes-devtools/pseudo/pseudo_1.6.2.bb | |||
@@ -3,6 +3,8 @@ require pseudo.inc | |||
3 | SRC_URI = " \ | 3 | SRC_URI = " \ |
4 | http://www.yoctoproject.org/downloads/${BPN}/${BPN}-${PV}.tar.bz2 \ | 4 | http://www.yoctoproject.org/downloads/${BPN}/${BPN}-${PV}.tar.bz2 \ |
5 | file://0001-pseudo_client.c-protect-pwd_lck-against-magic.patch \ | 5 | file://0001-pseudo_client.c-protect-pwd_lck-against-magic.patch \ |
6 | file://0002-pseudo_util-modify-interface-to-pseudo_etc_file.patch \ | ||
7 | file://0003-pseudo_client.c-support-multiple-directories-in-PSEU.patch \ | ||
6 | " | 8 | " |
7 | 9 | ||
8 | SRC_URI[md5sum] = "4d7b4f9d1b4aafa680ce94a5a9a52f1f" | 10 | SRC_URI[md5sum] = "4d7b4f9d1b4aafa680ce94a5a9a52f1f" |