All of lore.kernel.org
 help / color / mirror / Atom feed
* [OE-core] [PATCH] kernel-fitimage.bbclass: fix wrong separators
@ 2021-05-21 12:20 Ming Liu
  2021-05-21 14:28 ` Richard Purdie
  0 siblings, 1 reply; 5+ messages in thread
From: Ming Liu @ 2021-05-21 12:20 UTC (permalink / raw)
  To: openembedded-core; +Cc: Ming Liu

From: Ming Liu <liu.ming50@gmail.com>

Change to use '-' rather than '@' as separators.

Signed-off-by: Ming Liu <liu.ming50@gmail.com>
---
 meta/classes/kernel-fitimage.bbclass | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/classes/kernel-fitimage.bbclass b/meta/classes/kernel-fitimage.bbclass
index 5cfd8af99d..4858037706 100644
--- a/meta/classes/kernel-fitimage.bbclass
+++ b/meta/classes/kernel-fitimage.bbclass
@@ -217,13 +217,13 @@ fitimage_emit_section_boot_script() {
 	bootscr_sign_keyname="${UBOOT_SIGN_KEYNAME}"
 
         cat << EOF >> ${1}
-                bootscr@${2} {
+                bootscr-${2} {
                         description = "U-boot script";
                         data = /incbin/("${3}");
                         type = "script";
                         arch = "${UBOOT_ARCH}";
                         compression = "none";
-                        hash@1 {
+                        hash-1 {
                                 algo = "${bootscr_csum}";
                         };
                 };
@@ -232,7 +232,7 @@ EOF
 	if [ "${UBOOT_SIGN_ENABLE}" = "1" -a "${FIT_SIGN_INDIVIDUAL}" = "1" -a -n "${bootscr_sign_keyname}" ] ; then
 		sed -i '$ d' ${1}
 		cat << EOF >> ${1}
-                        signature@1 {
+                        signature-1 {
                                 algo = "${bootscr_csum},${bootscr_sign_algo}";
                                 key-name-hint = "${bootscr_sign_keyname}";
                         };
@@ -383,7 +383,7 @@ fitimage_emit_section_config() {
 	if [ -n "${bootscr_id}" ]; then
 		conf_desc="${conf_desc}${sep}u-boot script"
 		sep=", "
-		bootscr_line="bootscr = \"bootscr@${bootscr_id}\";"
+		bootscr_line="bootscr = \"bootscr-${bootscr_id}\";"
 	fi
 
 	if [ -n "${config_id}" ]; then
-- 
2.29.0


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [OE-core] [PATCH] kernel-fitimage.bbclass: fix wrong separators
  2021-05-21 12:20 [OE-core] [PATCH] kernel-fitimage.bbclass: fix wrong separators Ming Liu
@ 2021-05-21 14:28 ` Richard Purdie
  2021-05-21 14:44   ` Bruce Ashfield
  0 siblings, 1 reply; 5+ messages in thread
From: Richard Purdie @ 2021-05-21 14:28 UTC (permalink / raw)
  To: Ming Liu, openembedded-core

On Fri, 2021-05-21 at 14:20 +0200, Ming Liu wrote:
> From: Ming Liu <liu.ming50@gmail.com>
> 
> Change to use '-' rather than '@' as separators.

Why? The commit message needs more information.

Cheers,

Richard


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [OE-core] [PATCH] kernel-fitimage.bbclass: fix wrong separators
  2021-05-21 14:28 ` Richard Purdie
@ 2021-05-21 14:44   ` Bruce Ashfield
  2021-05-21 16:04     ` Andrey Zhizhikin
  0 siblings, 1 reply; 5+ messages in thread
From: Bruce Ashfield @ 2021-05-21 14:44 UTC (permalink / raw)
  To: Richard Purdie; +Cc: Ming Liu, Patches and discussions about the oe-core layer

On Fri, May 21, 2021 at 10:28 AM Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>
> On Fri, 2021-05-21 at 14:20 +0200, Ming Liu wrote:
> > From: Ming Liu <liu.ming50@gmail.com>
> >
> > Change to use '-' rather than '@' as separators.
>
> Why? The commit message needs more information.
>

I had exactly the same question/comment!

Bruce

> Cheers,
>
> Richard
>
>
> 
>


-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [OE-core] [PATCH] kernel-fitimage.bbclass: fix wrong separators
  2021-05-21 14:44   ` Bruce Ashfield
@ 2021-05-21 16:04     ` Andrey Zhizhikin
  2021-05-22  7:19       ` Ming Liu
  0 siblings, 1 reply; 5+ messages in thread
From: Andrey Zhizhikin @ 2021-05-21 16:04 UTC (permalink / raw)
  To: Bruce Ashfield
  Cc: Richard Purdie, Ming Liu,
	Patches and discussions about the oe-core layer

On Fri, May 21, 2021 at 4:44 PM Bruce Ashfield <bruce.ashfield@gmail.com> wrote:
>
> On Fri, May 21, 2021 at 10:28 AM Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> >
> > On Fri, 2021-05-21 at 14:20 +0200, Ming Liu wrote:
> > > From: Ming Liu <liu.ming50@gmail.com>
> > >
> > > Change to use '-' rather than '@' as separators.
> >
> > Why? The commit message needs more information.
> >
>
> I had exactly the same question/comment!

I guess this comes from the attempt to fix CVE-2021-27138, which was
already addressed in U-Boot with commit 3f04db891a35 ("image: Check
for unit addresses in FITs").

`@` is prohibied, so the `-` should be used instead.

I believe it would throw the build error, somethign like this:
| ./tools/mkimage: verify_header failed for FIT Image support with exit
| code 1

I do agree though - commit message should state this reason for this change.

>
> Bruce
>
> > Cheers,
> >
> > Richard
> >
> >
> >
> >
>
>
> --
> - Thou shalt not follow the NULL pointer, for chaos and madness await
> thee at its end
> - "Use the force Harry" - Gandalf, Star Trek II
>
> 
>


-- 
Regards,
Andrey.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [OE-core] [PATCH] kernel-fitimage.bbclass: fix wrong separators
  2021-05-21 16:04     ` Andrey Zhizhikin
@ 2021-05-22  7:19       ` Ming Liu
  0 siblings, 0 replies; 5+ messages in thread
From: Ming Liu @ 2021-05-22  7:19 UTC (permalink / raw)
  To: Andrey Zhizhikin
  Cc: Bruce Ashfield, Richard Purdie,
	Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 1498 bytes --]

Hi, Andrey:

Thanks, exactly as you explained, it's to fix CVE-2021-27138, will add more
commit messages.

//Ming Liu

Andrey Zhizhikin <andrey.z@gmail.com> 於 2021年5月21日 週五 下午6:04寫道:

> On Fri, May 21, 2021 at 4:44 PM Bruce Ashfield <bruce.ashfield@gmail.com>
> wrote:
> >
> > On Fri, May 21, 2021 at 10:28 AM Richard Purdie
> > <richard.purdie@linuxfoundation.org> wrote:
> > >
> > > On Fri, 2021-05-21 at 14:20 +0200, Ming Liu wrote:
> > > > From: Ming Liu <liu.ming50@gmail.com>
> > > >
> > > > Change to use '-' rather than '@' as separators.
> > >
> > > Why? The commit message needs more information.
> > >
> >
> > I had exactly the same question/comment!
>
> I guess this comes from the attempt to fix CVE-2021-27138, which was
> already addressed in U-Boot with commit 3f04db891a35 ("image: Check
> for unit addresses in FITs").
>
> `@` is prohibied, so the `-` should be used instead.
>
> I believe it would throw the build error, somethign like this:
> | ./tools/mkimage: verify_header failed for FIT Image support with exit
> | code 1
>
> I do agree though - commit message should state this reason for this
> change.
>
> >
> > Bruce
> >
> > > Cheers,
> > >
> > > Richard
> > >
> > >
> > >
> > >
> >
> >
> > --
> > - Thou shalt not follow the NULL pointer, for chaos and madness await
> > thee at its end
> > - "Use the force Harry" - Gandalf, Star Trek II
> >
> > 
> >
>
>
> --
> Regards,
> Andrey.
>

[-- Attachment #2: Type: text/html, Size: 2318 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-05-22  7:19 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-21 12:20 [OE-core] [PATCH] kernel-fitimage.bbclass: fix wrong separators Ming Liu
2021-05-21 14:28 ` Richard Purdie
2021-05-21 14:44   ` Bruce Ashfield
2021-05-21 16:04     ` Andrey Zhizhikin
2021-05-22  7:19       ` Ming Liu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.