Re: [PATCH] Replace HTTP links with HTTPS ones: drivers/iio

From: Jonathan Cameron <jic23@kernel.org>
To: "Alexander A. Klimov" <grandmaster@al2klimov.de>
Cc: lars@metafoo.de, Michael.Hennerich@analog.com, knaack.h@gmx.de,
	pmeerw@pmeerw.net, tomislav.denis@avl.com, ak@it-klinger.de,
	lukas@wunner.de, sergiu.cuciurean@analog.com,
	andriy.shevchenko@linux.intel.com, christophe.jaillet@wanadoo.fr,
	gregory.clement@bootlin.com, alexandre.belloni@bootlin.com,
	sre@kernel.org, matt.ranostay@konsulko.com, hslester96@gmail.com,
	chris.lesiak@licor.com, alexandru.ardelean@analog.com,
	dannenberg@ti.com, dpfrey@gmail.com, linux-iio@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Replace HTTP links with HTTPS ones: drivers/iio
Date: Sat, 4 Jul 2020 17:25:13 +0100	[thread overview]
Message-ID: <20200704172513.03815356@archlinux> (raw)
In-Reply-To: <20200703223124.16081-1-grandmaster@al2klimov.de>

On Sat,  4 Jul 2020 00:31:24 +0200
"Alexander A. Klimov" <grandmaster@al2klimov.de> 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 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@al2klimov.de>
> ---
>  Continuing my work started at 93431e0607e5.
> 
>  If there are any URLs to be removed completely or at least not 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
> 

I did a quick click through of the resulting links. 

> diff --git a/drivers/iio/humidity/si7020.c b/drivers/iio/humidity/si7020.c
> index b938f07eed64..612cecbf8e7e 100644
> --- a/drivers/iio/humidity/si7020.c
> +++ b/drivers/iio/humidity/si7020.c
> @@ -13,7 +13,7 @@
>   *
>   * Data Sheets:
>   *   Si7013: http://www.silabs.com/Support%20Documents/TechnicalDocs/Si7013.pdf
> - *   Si7020: http://www.silabs.com/Support%20Documents/TechnicalDocs/Si7020.pdf
> + *   Si7020: https://www.silabs.com/Support%20Documents/TechnicalDocs/Si7020.pdf
>   *   Si7021: http://www.silabs.com/Support%20Documents/TechnicalDocs/Si7021.pdf
The original and new link don't actually work. 

Seems they have changed the doc name. Not sure what the A20 reflects but there
doesn't seem to be any other variant of each of these.

https://www.silabs.com/documents/public/data-sheets/Si7013-A20.pdf
https://www.silabs.com/documents/public/data-sheets/Si7020-A20.pdf
https://www.silabs.com/documents/public/data-sheets/Si7021-A20.pdf

>   */
>  
> diff --git a/drivers/iio/light/isl29028.c b/drivers/iio/light/isl29028.c
> index 4d220c835c75..172fc9df1140 100644
> --- a/drivers/iio/light/isl29028.c
> +++ b/drivers/iio/light/isl29028.c
> @@ -8,7 +8,7 @@
>   *
>   * Datasheets:
>   *  - http://www.intersil.com/content/dam/Intersil/documents/isl2/isl29028.pdf
> - *  - http://www.intersil.com/content/dam/Intersil/documents/isl2/isl29030.pdf
> + *  - https://www.intersil.com/content/dam/Intersil/documents/isl2/isl29030.pdf

I'm not getting anywhere with this link.  Given intersil seems to have
be bought be renesas

https://www.renesas.com/eu/en/www/doc/datasheet/isl29028a.pdf (not 100% sure this is
compatible with the non a version.

Not finding anything on the isl29030 so probably best to just drop that reference.

>   */
>  
>  #include <linux/module.h>

The rest seem fine.  Thanks for doing this.

Jonathan