diff mbox series

[4/4] Documentation/admin-guide/module-signing.rst: add openssl command option example for CodeSign EKU

Message ID 20210222064251.13374-5-jlee@suse.com
State Superseded
Headers show
Series Check codeSigning extended key usage extension | expand

Commit Message

Lee, Chun-Yi Feb. 22, 2021, 6:42 a.m. UTC
Add an openssl command option example for generating CodeSign extended
key usage in X.509 when CONFIG_CHECK_CODESIGN_EKU is enabled.

Signed-off-by: "Lee, Chun-Yi" <jlee@suse.com>
---
 Documentation/admin-guide/module-signing.rst | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Randy Dunlap Feb. 22, 2021, 4:48 p.m. UTC | #1
Hi,

On 2/21/21 10:42 PM, Lee, Chun-Yi wrote:
> Add an openssl command option example for generating CodeSign extended
> key usage in X.509 when CONFIG_CHECK_CODESIGN_EKU is enabled.
> 
> Signed-off-by: "Lee, Chun-Yi" <jlee@suse.com>
> ---
>  Documentation/admin-guide/module-signing.rst | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/admin-guide/module-signing.rst b/Documentation/admin-guide/module-signing.rst
> index 7d7c7c8a545c..b57b30c7125f 100644
> --- a/Documentation/admin-guide/module-signing.rst
> +++ b/Documentation/admin-guide/module-signing.rst
> @@ -170,6 +170,12 @@ generate the public/private key files::
>  	   -config x509.genkey -outform PEM -out kernel_key.pem \
>  	   -keyout kernel_key.pem
>  
> +When ``CONFIG_CHECK_CODESIGN_EKU`` option be enabled, the following openssl

                                             is enabled,

> +command option should be added for generating CodeSign extended key usage in

              should be added <where>?

> +X.509::
> +
> +        -addext "extendedKeyUsage=codeSigning"
> +
>  The full pathname for the resulting kernel_key.pem file can then be specified
>  in the ``CONFIG_MODULE_SIG_KEY`` option, and the certificate and key therein will
>  be used instead of an autogenerated keypair.
> 

thanks.
Lee, Chun-Yi Feb. 24, 2021, 9:27 a.m. UTC | #2
Hi Randy,

On Mon, Feb 22, 2021 at 08:48:42AM -0800, Randy Dunlap wrote:
> Hi,

> 

> On 2/21/21 10:42 PM, Lee, Chun-Yi wrote:

> > Add an openssl command option example for generating CodeSign extended

> > key usage in X.509 when CONFIG_CHECK_CODESIGN_EKU is enabled.

> > 

> > Signed-off-by: "Lee, Chun-Yi" <jlee@suse.com>

> > ---

> >  Documentation/admin-guide/module-signing.rst | 6 ++++++

> >  1 file changed, 6 insertions(+)

> > 

> > diff --git a/Documentation/admin-guide/module-signing.rst b/Documentation/admin-guide/module-signing.rst

> > index 7d7c7c8a545c..b57b30c7125f 100644

> > --- a/Documentation/admin-guide/module-signing.rst

> > +++ b/Documentation/admin-guide/module-signing.rst

> > @@ -170,6 +170,12 @@ generate the public/private key files::

> >  	   -config x509.genkey -outform PEM -out kernel_key.pem \

> >  	   -keyout kernel_key.pem

> >  

> > +When ``CONFIG_CHECK_CODESIGN_EKU`` option be enabled, the following openssl

> 

>                                              is enabled,

> 

> > +command option should be added for generating CodeSign extended key usage in

> 

>               should be added <where>?

> 

> > +X.509::


Thanks for your review! I will update to next version.

Joey Lee
diff mbox series

Patch

diff --git a/Documentation/admin-guide/module-signing.rst b/Documentation/admin-guide/module-signing.rst
index 7d7c7c8a545c..b57b30c7125f 100644
--- a/Documentation/admin-guide/module-signing.rst
+++ b/Documentation/admin-guide/module-signing.rst
@@ -170,6 +170,12 @@  generate the public/private key files::
 	   -config x509.genkey -outform PEM -out kernel_key.pem \
 	   -keyout kernel_key.pem
 
+When ``CONFIG_CHECK_CODESIGN_EKU`` option be enabled, the following openssl
+command option should be added for generating CodeSign extended key usage in
+X.509::
+
+        -addext "extendedKeyUsage=codeSigning"
+
 The full pathname for the resulting kernel_key.pem file can then be specified
 in the ``CONFIG_MODULE_SIG_KEY`` option, and the certificate and key therein will
 be used instead of an autogenerated keypair.