[SRU B/D linux-signed] UBUNTU: fix vmlinuz-* permissions for opal signed kernels

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[SRU B/D linux-signed] UBUNTU: fix vmlinuz-* permissions for opal signed kernels

Thadeu Lima de Souza Cascardo-3
From: Seth Forshee <[hidden email]>

BugLink: https://bugs.launchpad.net/bugs/1843327

We're currently changing the permissions on the signature file
instead of the concatenated kernel+signature file which will be
installed. Fix this.

Signed-off-by: Seth Forshee <[hidden email]>
Signed-off-by: Thadeu Lima de Souza Cascardo <[hidden email]>
---
 debian/rules | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/debian/rules b/debian/rules
index 62c132c83ba4..f11e4455f907 100755
--- a/debian/rules
+++ b/debian/rules
@@ -49,10 +49,10 @@ override_dh_auto_build:
  done; \
  for s in *.opal.sig; do \
  [ ! -f "$$s" ] && continue; \
- chmod 600 "$$s"; \
  base=$$(echo "$$s" | sed -e 's/.opal.sig//'); \
- cat "$$base.opal" "$$s" >"../SIGNED/$$base";\
- done \
+ cat "$$base.opal" "$$s" >"../SIGNED/$$base"; \
+ chmod 600 "../SIGNED/$$base"; \
+ done; \
  )
 
 override_dh_auto_install:
--
2.20.1


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK: [SRU B/D linux-signed] UBUNTU: fix vmlinuz-* permissions for opal signed kernels

Kleber Souza
On 09.09.19 21:37, Thadeu Lima de Souza Cascardo wrote:

> From: Seth Forshee <[hidden email]>
>
> BugLink: https://bugs.launchpad.net/bugs/1843327
>
> We're currently changing the permissions on the signature file
> instead of the concatenated kernel+signature file which will be
> installed. Fix this.
>
> Signed-off-by: Seth Forshee <[hidden email]>
> Signed-off-by: Thadeu Lima de Souza Cascardo <[hidden email]>

Acked-by: Kleber Sacilotto de Souza <[hidden email]>

> ---
>  debian/rules | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/debian/rules b/debian/rules
> index 62c132c83ba4..f11e4455f907 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -49,10 +49,10 @@ override_dh_auto_build:
>   done; \
>   for s in *.opal.sig; do \
>   [ ! -f "$$s" ] && continue; \
> - chmod 600 "$$s"; \
>   base=$$(echo "$$s" | sed -e 's/.opal.sig//'); \
> - cat "$$base.opal" "$$s" >"../SIGNED/$$base";\
> - done \
> + cat "$$base.opal" "$$s" >"../SIGNED/$$base"; \
> + chmod 600 "../SIGNED/$$base"; \
> + done; \
>   )
>  
>  override_dh_auto_install:
>


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK: [SRU B/D linux-signed] UBUNTU: fix vmlinuz-* permissions for opal signed kernels

Sultan Alsawaf
In reply to this post by Thadeu Lima de Souza Cascardo-3
On Mon, Sep 09, 2019 at 04:37:19PM -0300, Thadeu Lima de Souza Cascardo wrote:

> From: Seth Forshee <[hidden email]>
>
> BugLink: https://bugs.launchpad.net/bugs/1843327
>
> We're currently changing the permissions on the signature file
> instead of the concatenated kernel+signature file which will be
> installed. Fix this.
>
> Signed-off-by: Seth Forshee <[hidden email]>
> Signed-off-by: Thadeu Lima de Souza Cascardo <[hidden email]>
> ---
>  debian/rules | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/debian/rules b/debian/rules
> index 62c132c83ba4..f11e4455f907 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -49,10 +49,10 @@ override_dh_auto_build:
>   done; \
>   for s in *.opal.sig; do \
>   [ ! -f "$$s" ] && continue; \
> - chmod 600 "$$s"; \
>   base=$$(echo "$$s" | sed -e 's/.opal.sig//'); \
> - cat "$$base.opal" "$$s" >"../SIGNED/$$base";\
> - done \
> + cat "$$base.opal" "$$s" >"../SIGNED/$$base"; \
> + chmod 600 "../SIGNED/$$base"; \
> + done; \
>   )
>  
>  override_dh_auto_install:
> --
> 2.20.1
>
>
> --
> kernel-team mailing list
> [hidden email]
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

Acked-by: Sultan Alsawaf <[hidden email]>

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

APPLIED: [SRU B/D linux-signed] UBUNTU: fix vmlinuz-* permissions for opal signed kernels

Kleber Souza
In reply to this post by Thadeu Lima de Souza Cascardo-3
On 09.09.19 21:37, Thadeu Lima de Souza Cascardo wrote:

> From: Seth Forshee <[hidden email]>
>
> BugLink: https://bugs.launchpad.net/bugs/1843327
>
> We're currently changing the permissions on the signature file
> instead of the concatenated kernel+signature file which will be
> installed. Fix this.
>
> Signed-off-by: Seth Forshee <[hidden email]>
> Signed-off-by: Thadeu Lima de Souza Cascardo <[hidden email]>
> ---
>  debian/rules | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/debian/rules b/debian/rules
> index 62c132c83ba4..f11e4455f907 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -49,10 +49,10 @@ override_dh_auto_build:
>   done; \
>   for s in *.opal.sig; do \
>   [ ! -f "$$s" ] && continue; \
> - chmod 600 "$$s"; \
>   base=$$(echo "$$s" | sed -e 's/.opal.sig//'); \
> - cat "$$base.opal" "$$s" >"../SIGNED/$$base";\
> - done \
> + cat "$$base.opal" "$$s" >"../SIGNED/$$base"; \
> + chmod 600 "../SIGNED/$$base"; \
> + done; \
>   )
>  
>  override_dh_auto_install:
>

Applied to bionic/linux-signed/master and disco/linux-signed/master
branches.

Thanks,
Kleber

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team