Skip to content

Commit

Permalink
tree-wide: replace problematic terminology
Browse files Browse the repository at this point in the history
Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>
  • Loading branch information
Christian Brauner committed Jun 29, 2021
1 parent 53dfebf commit 111277a
Show file tree
Hide file tree
Showing 10 changed files with 18 additions and 18 deletions.
2 changes: 1 addition & 1 deletion doc/examples/lxc-complex.conf.in
Expand Up @@ -17,7 +17,7 @@ lxc.net.0.ipv6.address = 2003:db8:1:0:214:1234:fe0b:3596

lxc.net.0.type = phys
lxc.net.0.flags = up
lxc.net.0.link = dummy0
lxc.net.0.link = random0
lxc.net.0.hwaddr = 4a:49:43:49:79:ff
lxc.net.0.ipv4.address = 10.2.3.6/24
lxc.net.0.ipv6.address = 2003:db8:1:0:214:1234:fe0b:3297
2 changes: 1 addition & 1 deletion doc/ja/lxc.container.conf.sgml.in
Expand Up @@ -3730,7 +3730,7 @@ by KATOH Yasufumi <karma at jazz.email.ne.jp>
lxc.net.1.ipv6.address = 2003:db8:1:0:214:1234:fe0b:3596
lxc.net.2.type = phys
lxc.net.2.flags = up
lxc.net.2.link = dummy0
lxc.net.2.link = random0
lxc.net.2.hwaddr = 4a:49:43:49:79:ff
lxc.net.2.ipv4.address = 10.2.3.6/24
lxc.net.2.ipv6.address = 2003:db8:1:0:214:1234:fe0b:3297
Expand Down
2 changes: 1 addition & 1 deletion doc/ko/lxc.container.conf.sgml.in
Expand Up @@ -2613,7 +2613,7 @@ mknod errno 0
lxc.net.1.ipv6.address = 2003:db8:1:0:214:1234:fe0b:3596
lxc.net.2.type = phys
lxc.net.2.flags = up
lxc.net.2.link = dummy0
lxc.net.2.link = random0
lxc.net.2.hwaddr = 4a:49:43:49:79:ff
lxc.net.2.ipv4.address = 10.2.3.6/24
lxc.net.2.ipv6.address = 2003:db8:1:0:214:1234:fe0b:3297
Expand Down
2 changes: 1 addition & 1 deletion doc/lxc.container.conf.sgml.in
Expand Up @@ -3008,7 +3008,7 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
lxc.net.1.ipv6.address = 2003:db8:1:0:214:1234:fe0b:3596
lxc.net.2.type = phys
lxc.net.2.flags = up
lxc.net.2.link = dummy0
lxc.net.2.link = random0
lxc.net.2.hwaddr = 4a:49:43:49:79:ff
lxc.net.2.ipv4.address = 10.2.3.6/24
lxc.net.2.ipv6.address = 2003:db8:1:0:214:1234:fe0b:3297
Expand Down
4 changes: 2 additions & 2 deletions src/lxc/attach.c
Expand Up @@ -997,7 +997,7 @@ static char *lxc_attach_getpwshell(uid_t uid)
if (!token)
continue;

/* next: dummy password field */
/* next: placeholder password field */
token = strtok_r(NULL, ":", &saveptr);
if (!token)
continue;
Expand All @@ -1009,7 +1009,7 @@ static char *lxc_attach_getpwshell(uid_t uid)
value == LONG_MAX)
continue;

/* dummy sanity check: user id matches */
/* placeholder conherence check: user id matches */
if ((uid_t)value != uid)
continue;

Expand Down
4 changes: 2 additions & 2 deletions src/lxc/cgroups/cgroup2_devices.c
Expand Up @@ -528,7 +528,7 @@ int bpf_list_add_device(struct bpf_devices *bpf_devices,
bool bpf_devices_cgroup_supported(void)
{
__do_bpf_program_free struct bpf_program *prog = NULL;
const struct bpf_insn dummy[] = {
const struct bpf_insn insn[] = {
BPF_MOV64_IMM(BPF_REG_0, 1),
BPF_EXIT_INSN(),
};
Expand All @@ -546,7 +546,7 @@ bool bpf_devices_cgroup_supported(void)
if (ret)
return log_error_errno(false, ENOMEM, "Failed to initialize bpf program");

ret = bpf_program_add_instructions(prog, dummy, ARRAY_SIZE(dummy));
ret = bpf_program_add_instructions(prog, insn, ARRAY_SIZE(insn));
if (ret < 0)
return log_trace(false, "Failed to add new instructions to bpf device cgroup program");

Expand Down
10 changes: 5 additions & 5 deletions src/lxc/conf.c
Expand Up @@ -1722,11 +1722,11 @@ static int lxc_setup_devpts_child(struct lxc_handler *handler)
DEBUG("Removed existing \"/dev/ptmx\" file");
}

/* Create dummy /dev/ptmx file as bind mountpoint for /dev/pts/ptmx. */
/* Create placeholder /dev/ptmx file as bind mountpoint for /dev/pts/ptmx. */
ret = mknodat(rootfs->dfd_dev, "ptmx", S_IFREG | 0000, 0);
if (ret < 0 && errno != EEXIST)
return log_error_errno(-1, errno, "Failed to create dummy \"/dev/ptmx\" file as bind mount target");
DEBUG("Created dummy \"/dev/ptmx\" file as bind mount target");
return log_error_errno(-1, errno, "Failed to create \"/dev/ptmx\" file as bind mount target");
DEBUG("Created \"/dev/ptmx\" file as bind mount target");

/* Fallback option: create symlink /dev/ptmx -> /dev/pts/ptmx */
ret = mount("/dev/pts/ptmx", "/dev/ptmx", NULL, MS_BIND, NULL);
Expand All @@ -1736,7 +1736,7 @@ static int lxc_setup_devpts_child(struct lxc_handler *handler)
/* Fallthrough and try to create a symlink. */
ERROR("Failed to bind mount \"/dev/pts/ptmx\" to \"/dev/ptmx\"");

/* Remove the dummy /dev/ptmx file we created above. */
/* Remove the placeholder /dev/ptmx file we created above. */
ret = unlinkat(rootfs->dfd_dev, "ptmx", 0);
if (ret < 0)
return log_error_errno(-1, errno, "Failed to remove existing \"/dev/ptmx\"");
Expand Down Expand Up @@ -3761,7 +3761,7 @@ static int lxc_execute_bind_init(struct lxc_handler *handler)
if (!file_exists(destpath)) {
ret = mknod(destpath, S_IFREG | 0000, 0);
if (ret < 0 && errno != EEXIST)
return log_error_errno(-1, errno, "Failed to create dummy \"%s\" file as bind mount target", destpath);
return log_error_errno(-1, errno, "Failed to create \"%s\" file as bind mount target", destpath);
}

ret = safe_mount(path, destpath, "none", MS_BIND, NULL, conf->rootfs.mount);
Expand Down
6 changes: 3 additions & 3 deletions src/lxc/start.c
Expand Up @@ -1537,9 +1537,9 @@ int resolve_clone_flags(struct lxc_handler *handler)
* newer glibc versions where the getpid() cache is removed and the pid/tid is
* not reset anymore.
* However, if for whatever reason you - dear committer - somehow need to get the
* pid of the dummy intermediate process for do_share_ns() you need to call
* lxc_raw_getpid(). The next lxc_raw_clone() call does not employ CLONE_VM and
* will be fine.
* pid of the placeholder intermediate process for do_share_ns() you need to
* call lxc_raw_getpid(). The next lxc_raw_clone() call does not employ
* CLONE_VM and will be fine.
*/
static inline int do_share_ns(void *arg)
{
Expand Down
2 changes: 1 addition & 1 deletion src/lxc/tools/lxc_ls.c
Expand Up @@ -198,7 +198,7 @@ int main(int argc, char *argv[])

/*
* The lxc parser requires that my_args.name is set. So let's satisfy
* that condition by setting a dummy name which is never used.
* that condition by setting a placeholder name which is never used.
*/
my_args.name = "";
if (lxc_arguments_parse(&my_args, argc, argv))
Expand Down
2 changes: 1 addition & 1 deletion src/tests/lxc-test-unpriv
Expand Up @@ -42,7 +42,7 @@ if modprobe -q overlayfs; then
mkdir ${MOUNTDIR}/{lowerdir,upperdir,workdir,overlayfs}
mount -t overlayfs -o lowerdir="${MOUNTDIR}/lowerdir",upperdir="${MOUNTDIR}/upperdir",workdir="${MOUNTDIR}/workdir" none "${MOUNTDIR}/overlayfs"

CORRECT_LINK_TARGET="${MOUNTDIR}/overlayfs/dummy_file"
CORRECT_LINK_TARGET="${MOUNTDIR}/overlayfs/placeholder_file"
exec 9> "${CORRECT_LINK_TARGET}"

DETECTED_LINK_TARGET=$(readlink -q /proc/$$/fd/9)
Expand Down

0 comments on commit 111277a

Please sign in to comment.