rhboot / shim-review

Reviews of shim
66 stars 127 forks source link

Shim-15.8 for LUX 2.0 x64_ia32 #396

Open Rodrigo-NR opened 6 months ago

Rodrigo-NR commented 6 months ago

Confirm the following are included in your repo, checking each box:


What is the link to your tag in a repo cloned from rhboot/shim-review?


https://github.com/Rodrigo-NR/shim-review/tree/lux2.0-shim-amd64_i386-20240304

New tag with GRUB 1+2.12+2: https://github.com/Rodrigo-NR/shim-review/tree/lux2.0-shim-amd64_i386-20240612


What is the SHA256 hash of your final SHIM binary?


7e8e4368bb69563d5c479fe61270ceb4fe61e9dc06575e4645426713590aa9da shimia32.efi c2afb5e3c305c894c299b54157a1a05891e4b7b0f6722a00696999820490e5db shimx64.efi


What is the link to your previous shim review request (if any, otherwise N/A)?


https://github.com/rhboot/shim-review/issues/308

SherifNagy commented 6 months ago

Just a quick note, you might want to increase grub.debian entry to 5 and release a new grub or wait until Debian releases new group with the increased global generation number, current shim 15.8 already revokes grub.debian,4

THS-on commented 6 months ago

@Rodrigo-NR I see that the second security contact changed to you and it is with an IPT email address. Because you are submitting on behalf of Lenovo, can @ManigaLenovo and you clarify what the relationship these two entities have?

ManigaLenovo commented 6 months ago

Lenovo has an R&D contract with IPT, and we are developing the Lux Linux distribution. @Rodrigo-NR is the IPT developer responsible for secure boot And @icteixeira works with us in the development of Lux Linux distro.

Below is the link where you can check more about Lenovo and IPT partnership (only in portuguese).

https://ipt.br/2023/05/23/tecnologias-digitais-para-todos/

es-fabricemarie commented 5 months ago

I am not an official reviewer, but I just want to help with reviewers workload

dennis-tseng99 commented 5 months ago

Just briefly reviewing :

Rodrigo-NR commented 4 months ago

Thank you for the reviews. We will await feedback from the official reviewers regarding the certificate currently in use. I have not received the contact verification email, should we take any further action?

es-fabricemarie commented 4 months ago

@Rodrigo-NR If you plan on using your distro only on Lenovos and if you sign the shim yourself using that certificate it should just work (i.e. no need to review it and get it signed by Microsoft CA). Give it a try :)

SherifNagy commented 4 months ago

Just a quick note, you might want to increase grub.debian entry to 5 and release a new grub or wait until Debian releases new group with the increased global generation number, current shim 15.8 already revokes grub.debian,4

By the way, seems like there was a bit a mistake here, you can keep the grub.debian,4 , there is no harm if you already released the grub,debian,5 , just thought to mention it.

THS-on commented 3 months ago

Review of lux2.0-shim-amd64_i386-20240304

Shim

GRUB2 and fwupd

Linux Kernel

Notes and Questions

steve-mcintyre commented 3 months ago

Contact verification emails sent - please respond here as instructed.

icteixeira commented 3 months ago

Contact verification emails sent - please respond here as instructed.

alighting spooking bedroll cascade hampered mismatches oxygenates funnest a= pprehensively suborns

Rodrigo-NR commented 3 months ago

Contact verification emails sent - please respond here as instructed.

stupefy Panamanian squalor cleverly oversimplified racoon hypo rinses curls= drainpipes

steve-mcintyre commented 3 months ago

Two contacts have responded, verification is good

steve-mcintyre commented 3 months ago

Notes and Questions

* Looking at other CAs, the Debian's CA is 2048 bit and valid until 2046. @steve-mcintyre any opinion on this?

We discussed this in our meeting last week too. There's a worry that lots of firmware implementations still won't do more than 2048-bit RSA safely. :-( 2048 will do for now, and we'll check on that periodically.

* Please update to GRUB2 2.12-2 as it fixes a CVE in the peimage loader

Definitely this is needed

* Are your kernel sources somewhere public?

Do you have an answer for this please?

Rodrigo-NR commented 3 months ago

Hello, @THS-on and @steve-mcintyre

We have updated GRUB, based on version 2+2.12+2 from Debian, and also updated SBAT. The modifications are in the tag https://github.com/Rodrigo-NR/shim-review/tree/lux2.0-shim-amd64_i386-20240612/.

The kernel source is available at https://github.com/rcilto1/kernel. We used version 6.6.18 from Kernel.org and applied some patches that are in the respective folder.

THS-on commented 3 months ago

@Rodrigo-NR thanks!

Please also add the new tag to the top comment of this issue, otherwise LGTM from my side. @dennis-tseng99 @steve-mcintyre can one of you have a look again?

Rodrigo-NR commented 3 months ago

Thank you @THS-on,

I added the new tag in the top comment.

steve-mcintyre commented 2 months ago

Review of Shim-15.8 for LUX 2.0 x64_ia32

OK

Issues / queries

None!

All looks good, accepting!

THS-on commented 1 month ago

@Rodrigo-NR did you get a signed shim back?

Rodrigo-NR commented 1 month ago

@THS-on We submitted the SHIM to Microsoft, but it was not accepted and we don't know why. We are in contact with them to understand the reason for the rejection from Microsoft's side.

es-fabricemarie commented 1 month ago

@Rodrigo-NR could it be that it is because your certificate is already in all Lenovo's keys in the UEFI DB keyring (as mentioned in my previous comment)? As mentioned, on my Lenovo laptop I already have cert with serial number 03:09:48:62:90:34:75:92:87:34:95:87:23:09:4d in my machine's DB keyring.

Otherwise are you sure that you:

  1. packaged your efi binaries in a cab file
  2. without directories
  3. signed the resulting cab file with your EV Code Signing certificate
THS-on commented 1 day ago

@Rodrigo-NR what is the state of the submission with Microsoft?