Re: [PATCH for v5.9] ARM: STM32: Replace HTTP links with HTTPS ones

[Alexander A. Klimov]

Am 21.07.20 um 10:49 schrieb Alexandre Torgue:
> Hi Alexander
>
> On 7/19/20 11:49 AM, Alexander A. Klimov wrote:
> > Rationale:
> > Reduces attack surface on kernel devs opening the links for MITM
> > as HTTPS traffic is much harder to manipulate.
> >
> > Deterministic algorithm:
> > For each file:
> > ÂÂ If not .svg:
> > ÂÂÂÂ For each line:
> > ÂÂÂÂÂÂ If doesn't contain `\bxmlns\b`:
> > ÂÂÂÂÂÂÂÂ For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:
> > ÂÂÂÂÂ If neither `\bgnu\.org/license`, nor `\bmozilla\.org/MPL\b`:
> > ÂÂÂÂÂÂÂÂÂÂÂÂ If both the HTTP and HTTPS versions
> > ÂÂÂÂÂÂÂÂÂÂÂÂ return 200 OK and serve the same content:
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂ Replace HTTP with HTTPS.
> >
> > Signed-off-by: Alexander A. Klimov <grandmaster@xxxxxxxxxxxx>
>
> This patch touch 2 different subsystems. Can you please split it ?
I can. But don't all files belong to the subsystem this patch is for?

â  linux git:(autogen/1029) git show arch/arm/mach-stm32/Makefile.boot 
|perl scripts/get_maintainer.pl --nogit{,-fallback}
Russell King <linux@xxxxxxxxxxxxxxx> (odd fixer:ARM PORT)
Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx> (maintainer:ARM/STM32 
ARCHITECTURE)
Alexandre Torgue <alexandre.torgue@xxxxxx> (maintainer:ARM/STM32 
ARCHITECTURE)
linux-arm-kernel@xxxxxxxxxxxxxxxxxxx (moderated list:ARM SUB-ARCHITECTURES)
linux-stm32@xxxxxxxxxxxxxxxxxxxxxxxxxxxx (moderated list:ARM/STM32 
ARCHITECTURE)
linux-kernel@xxxxxxxxxxxxxxx (open list)
â  linux git:(autogen/1029) git show crypto/testmgr.h |perl 
scripts/get_maintainer.pl --nogit{,-fallback}
Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> (maintainer:CRYPTO API)
"David S. Miller" <davem@xxxxxxxxxxxxx> (maintainer:CRYPTO API)
Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx> (maintainer:ARM/STM32 
ARCHITECTURE)
Alexandre Torgue <alexandre.torgue@xxxxxx> (maintainer:ARM/STM32 
ARCHITECTURE)
linux-crypto@xxxxxxxxxxxxxxx (open list:CRYPTO API)
linux-stm32@xxxxxxxxxxxxxxxxxxxxxxxxxxxx (moderated list:ARM/STM32 
ARCHITECTURE)
linux-arm-kernel@xxxxxxxxxxxxxxxxxxx (moderated list:ARM/STM32 ARCHITECTURE)
linux-kernel@xxxxxxxxxxxxxxx (open list)
â  linux git:(autogen/1029)

> Regards
> Alex
>
>
> > ---
> > Â Continuing my work started at 93431e0607e5.
> > Â See also: git log --oneline '--author=Alexander A. Klimov 
> > <grandmaster@xxxxxxxxxxxx>' v5.7..master
> > Â (Actually letting a shell for loop submit all this stuff for me.)
> >
> > Â If there are any URLs to be removed completely
> > Â or at least not (just) HTTPSified:
> > Â Just clearly say so and I'll *undo my change*.
> > Â See also: https://lkml.org/lkml/2020/6/27/64
> >
> > Â If there are any valid, but yet not changed URLs:
> > Â See: https://lkml.org/lkml/2020/6/26/837
> >
> > Â If you apply the patch, please let me know.
> >
> > Â Sorry again to all maintainers who complained about subject lines.
> > Â Now I realized that you want an actually perfect prefixes,
> > Â not just subsystem ones.
> > Â I tried my best...
> > Â And yes, *I could* (at least half-)automate it.
> > Â Impossible is nothing! :)
> >
> >
> > Â arch/arm/mach-stm32/Makefile.boot | 2 +-
> > Â crypto/testmgr.hÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ | 6 +++---
> > Â 2 files changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm/mach-stm32/Makefile.boot 
> > b/arch/arm/mach-stm32/Makefile.boot
> > index cec195d4fcba..5dde7328a7a9 100644
> > --- a/arch/arm/mach-stm32/Makefile.boot
> > +++ b/arch/arm/mach-stm32/Makefile.boot
> > @@ -1,4 +1,4 @@
> > Â # SPDX-License-Identifier: GPL-2.0-only
> > Â # Empty file waiting for deletion once Makefile.boot isn't needed 
> > any more.
> > Â # Patch waits for application at
> > -# 
> > http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=7889/1 .
> > +# 
> > https://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=7889/1 .
> > diff --git a/crypto/testmgr.h b/crypto/testmgr.h
> > index d29983908c38..cdcf0d2fe40d 100644
> > --- a/crypto/testmgr.h
> > +++ b/crypto/testmgr.h
> > @@ -16231,7 +16231,7 @@ static const struct cipher_testvec 
> > aes_lrw_tv_template[] = {
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ "\xe9\x5d\x48\x92\x54\x63\x4e\xb8",
> > ÂÂÂÂÂÂÂÂÂ .lenÂÂÂ = 48,
> > ÂÂÂÂÂ }, {
> > -/* 
> > http://www.mail-archive.com/stds-p1619@xxxxxxxxxxxxxxxxx/msg00173.html */
> > +/* 
> > https://www.mail-archive.com/stds-p1619@xxxxxxxxxxxxxxxxx/msg00173.html */ 
> >
> > ÂÂÂÂÂÂÂÂÂ .keyÂÂÂ = "\xf8\xd4\x76\xff\xd6\x46\xee\x6c"
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ "\x23\x84\xcb\x1c\x77\xd6\x19\x5d"
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ "\xfe\xf1\xa9\xf3\x7b\xbc\x8d\x21"
> > @@ -21096,7 +21096,7 @@ static const struct aead_testvec 
> > aegis128_tv_template[] = {
> > Â /*
> > ÂÂ * All key wrapping test vectors taken from
> > - * http://csrc.nist.gov/groups/STM/cavp/documents/mac/kwtestvectors.zip
> > + * https://csrc.nist.gov/groups/STM/cavp/documents/mac/kwtestvectors.zip
> > ÂÂ *
> > ÂÂ * Note: as documented in keywrap.c, the ivout for encryption is the 
> > first
> > ÂÂ * semiblock of the ciphertext from the test vector. For decryption, 
> > iv is
> > @@ -22825,7 +22825,7 @@ static const struct cipher_testvec 
> > xeta_tv_template[] = {
> > ÂÂ * FCrypt test vectors
> > ÂÂ */
> > Â static const struct cipher_testvec fcrypt_pcbc_tv_template[] = {
> > -ÂÂÂ { /* 
> > http://www.openafs.org/pipermail/openafs-devel/2000-December/005320.html 
> > */
> > +ÂÂÂ { /* 
> > https://www.openafs.org/pipermail/openafs-devel/2000-December/005320.html 
> > */
> > ÂÂÂÂÂÂÂÂÂ .keyÂÂÂ = "\x00\x00\x00\x00\x00\x00\x00\x00",
> > ÂÂÂÂÂÂÂÂÂ .klenÂÂÂ = 8,
> > ÂÂÂÂÂÂÂÂÂ .ivÂÂÂ = "\x00\x00\x00\x00\x00\x00\x00\x00",