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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





Am 22.07.20 um 08:54 schrieb Lee Jones:
On Tue, 21 Jul 2020, Rob Herring wrote:

On Sun, 19 Jul 2020 21:56:13 +0200, 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>
---
  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.)

I'm guessing something went wrong with your submission, as Rob's reply
is the first time I saw this patch.  Did your bot send it to me?
Yes,
https://lore.kernel.org/lkml/20200719195613.61458-1-grandmaster@xxxxxxxxxxxx/


You really should be fully reviewing anything that gets sent to the
MLs.  "Sorry, I didn't send it, my bot did" is not an acceptable
excuse for inadequacies.
See this as an excuse for not stopping sending on demand (if any), not for invalid To:. Whoever's (not) in To: of auto-submitted patches would also (not) be there on manual submission because the bot just emulates myself:


➜  linux git:(master) cat submit.sh
#!/bin/bash
set -exo pipefail

if [ "$(perl scripts/get_maintainer.pl --nogit{,-fallback} --nol "$1" |wc -l)" -eq 0 ]; then
        GCP=--git-chief-penguins
else
        GCP=''
fi

git send-email --confirm=never "--to=$(
        perl scripts/get_maintainer.pl $GCP --norolestats "$1" |\
        perl -pe 'if (/<(.+?)>/) { $_ = "$1\n" }' |\
        tr \\n , |\
        perl -pe 's/,$//'
)" "$@"


  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! :)


  Documentation/devicetree/bindings/mfd/twl-family.txt | 2 +-
  drivers/mfd/hi6421-pmic-core.c                       | 2 +-
  drivers/mfd/lp873x.c                                 | 2 +-
  drivers/mfd/lp87565.c                                | 2 +-
  drivers/mfd/omap-usb-host.c                          | 2 +-
  drivers/mfd/omap-usb-tll.c                           | 2 +-
  drivers/mfd/smsc-ece1099.c                           | 2 +-
  drivers/mfd/ti_am335x_tscadc.c                       | 2 +-
  drivers/mfd/tps65086.c                               | 2 +-
  drivers/mfd/tps65217.c                               | 2 +-
  drivers/mfd/tps65218.c                               | 2 +-
  drivers/mfd/tps65912-core.c                          | 2 +-
  drivers/mfd/tps65912-i2c.c                           | 2 +-
  drivers/mfd/tps65912-spi.c                           | 2 +-
  include/linux/mfd/hi6421-pmic.h                      | 2 +-
  include/linux/mfd/lp873x.h                           | 2 +-
  include/linux/mfd/lp87565.h                          | 2 +-
  include/linux/mfd/ti_am335x_tscadc.h                 | 2 +-
  include/linux/mfd/tps65086.h                         | 2 +-
  include/linux/mfd/tps65217.h                         | 2 +-
  include/linux/mfd/tps65218.h                         | 2 +-
  include/linux/mfd/tps65912.h                         | 2 +-
  22 files changed, 22 insertions(+), 22 deletions(-)


Acked-by: Rob Herring <robh@xxxxxxxxxx>




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux