Skip to content

Misc improvements#15681

Merged
tomponline merged 3 commits into
canonical:mainfrom
simondeziel:misc-improvements
May 28, 2025
Merged

Misc improvements#15681
tomponline merged 3 commits into
canonical:mainfrom
simondeziel:misc-improvements

Conversation

@simondeziel
Copy link
Copy Markdown
Member

No description provided.

Adding the `/etc/inittab` was a workaround added in 2015 but it seems to not be
required anymore. Also, the workaround used a leading `/` which annoyed `tar`
that needed to strip it to avoid path traversal attacks.

```
$ tar tvf c50b454b13f4f017b067e047e08ec53a19eb3ec9f7e4310ff9cb89a0b4a96127.tar.xz
...
drw-r--r-- 0/0               0 1969-12-31 19:00 rootfs/tmp/
drw-r--r-- 0/0               0 1969-12-31 19:00 templates/
-rw-r--r-- 0/0             236 1969-12-31 19:00 templates/template.tpl
-rw-r--r-- 0/0             380 1969-12-31 19:00 metadata.yaml
tar: Removing leading `/' from member names
-rw-r--r-- 0/0               1 1969-12-31 19:00 /rootfs/etc/inittab
```

Signed-off-by: Simon Deziel <simon.deziel@canonical.com>

test/deps/import-busybox: use HTTPS link

Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
…egments

Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
@simondeziel simondeziel marked this pull request as ready for review May 28, 2025 21:55
@simondeziel simondeziel requested a review from tomponline May 28, 2025 21:55
@tomponline tomponline merged commit 9ad5d65 into canonical:main May 28, 2025
30 checks passed
@simondeziel simondeziel deleted the misc-improvements branch May 29, 2025 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants