devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores
@ 2020-07-05  7:55 Alexander A. Klimov
  2020-07-08 10:48 ` Michal Simek
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Alexander A. Klimov @ 2020-07-05  7:55 UTC (permalink / raw)
  To: hyun.kwon, laurent.pinchart, mchehab, robh+dt, michal.simek,
	linux-media, devicetree, linux-arm-kernel, linux-kernel
  Cc: Alexander A. Klimov

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

 Documentation/devicetree/bindings/media/xilinx/video.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/media/xilinx/video.txt b/Documentation/devicetree/bindings/media/xilinx/video.txt
index 68ac210e688e..d0335ca0cd57 100644
--- a/Documentation/devicetree/bindings/media/xilinx/video.txt
+++ b/Documentation/devicetree/bindings/media/xilinx/video.txt
@@ -32,4 +32,4 @@ The following properties are common to all Xilinx video IP cores.
   defaults to "mono".
 
 
-[UG934] http://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf
+[UG934] https://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf
-- 
2.27.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores
  2020-07-05  7:55 [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores Alexander A. Klimov
@ 2020-07-08 10:48 ` Michal Simek
  2020-07-14 23:57 ` Laurent Pinchart
  2020-07-15 21:44 ` Rob Herring
  2 siblings, 0 replies; 5+ messages in thread
From: Michal Simek @ 2020-07-08 10:48 UTC (permalink / raw)
  To: Alexander A. Klimov, hyun.kwon, laurent.pinchart, mchehab,
	robh+dt, michal.simek, linux-media, devicetree, linux-arm-kernel,
	linux-kernel



On 05. 07. 20 9:55, 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 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
> 
>  Documentation/devicetree/bindings/media/xilinx/video.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/media/xilinx/video.txt b/Documentation/devicetree/bindings/media/xilinx/video.txt
> index 68ac210e688e..d0335ca0cd57 100644
> --- a/Documentation/devicetree/bindings/media/xilinx/video.txt
> +++ b/Documentation/devicetree/bindings/media/xilinx/video.txt
> @@ -32,4 +32,4 @@ The following properties are common to all Xilinx video IP cores.
>    defaults to "mono".
>  
>  
> -[UG934] http://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf
> +[UG934] https://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf
> 

I have no problem with this change.

Acked-by: Michal Simek <michal.simek@xilinx.com>

Thanks,
Michal

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores
  2020-07-05  7:55 [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores Alexander A. Klimov
  2020-07-08 10:48 ` Michal Simek
@ 2020-07-14 23:57 ` Laurent Pinchart
  2020-07-15  6:33   ` Alexander A. Klimov
  2020-07-15 21:44 ` Rob Herring
  2 siblings, 1 reply; 5+ messages in thread
From: Laurent Pinchart @ 2020-07-14 23:57 UTC (permalink / raw)
  To: Alexander A. Klimov
  Cc: hyun.kwon, mchehab, robh+dt, michal.simek, linux-media,
	devicetree, linux-arm-kernel, linux-kernel

Hi Alexander,

Thank you for the patch.

The subject line should be

dt-bindings: media: xilinx: Replace HTTP links with HTTPS ones

On Sun, Jul 05, 2020 at 09:55:24AM +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 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>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Will you get this patch merged as part of a bigger series, or do you
expect us to take care of it ? In the latter case, if you agree with the
subject line change, there's no need to resubmit, I can fix this when
applying the patch.

> ---
>  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
> 
>  Documentation/devicetree/bindings/media/xilinx/video.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/media/xilinx/video.txt b/Documentation/devicetree/bindings/media/xilinx/video.txt
> index 68ac210e688e..d0335ca0cd57 100644
> --- a/Documentation/devicetree/bindings/media/xilinx/video.txt
> +++ b/Documentation/devicetree/bindings/media/xilinx/video.txt
> @@ -32,4 +32,4 @@ The following properties are common to all Xilinx video IP cores.
>    defaults to "mono".
>  
>  
> -[UG934] http://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf
> +[UG934] https://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf

-- 
Regards,

Laurent Pinchart

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores
  2020-07-14 23:57 ` Laurent Pinchart
@ 2020-07-15  6:33   ` Alexander A. Klimov
  0 siblings, 0 replies; 5+ messages in thread
From: Alexander A. Klimov @ 2020-07-15  6:33 UTC (permalink / raw)
  To: Laurent Pinchart
  Cc: hyun.kwon, mchehab, robh+dt, michal.simek, linux-media,
	devicetree, linux-arm-kernel, linux-kernel



Am 15.07.20 um 01:57 schrieb Laurent Pinchart:
> Hi Alexander,
> 
> Thank you for the patch.
> 
> The subject line should be
> 
> dt-bindings: media: xilinx: Replace HTTP links with HTTPS ones
> 
> On Sun, Jul 05, 2020 at 09:55:24AM +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 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>
> 
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> 
> Will you get this patch merged as part of a bigger series, or do you
> expect us to take care of it ? In the latter case, if you agree with the
I've just followed the patch submission instructions as for any 
non-treewide patch. So yes, I assume the people I write to take care of 
the patch.

> subject line change, there's no need to resubmit, I can fix this when
> applying the patch.
OK.

> 
>> ---
>>   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
>>
>>   Documentation/devicetree/bindings/media/xilinx/video.txt | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/media/xilinx/video.txt b/Documentation/devicetree/bindings/media/xilinx/video.txt
>> index 68ac210e688e..d0335ca0cd57 100644
>> --- a/Documentation/devicetree/bindings/media/xilinx/video.txt
>> +++ b/Documentation/devicetree/bindings/media/xilinx/video.txt
>> @@ -32,4 +32,4 @@ The following properties are common to all Xilinx video IP cores.
>>     defaults to "mono".
>>   
>>   
>> -[UG934] http://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf
>> +[UG934] https://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores
  2020-07-05  7:55 [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores Alexander A. Klimov
  2020-07-08 10:48 ` Michal Simek
  2020-07-14 23:57 ` Laurent Pinchart
@ 2020-07-15 21:44 ` Rob Herring
  2 siblings, 0 replies; 5+ messages in thread
From: Rob Herring @ 2020-07-15 21:44 UTC (permalink / raw)
  To: Alexander A. Klimov
  Cc: devicetree, hyun.kwon, michal.simek, mchehab, robh+dt,
	linux-kernel, linux-arm-kernel, linux-media, laurent.pinchart

On Sun, 05 Jul 2020 09:55:24 +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 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
> 
>  Documentation/devicetree/bindings/media/xilinx/video.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Applied, thanks!

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-07-15 21:44 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-05  7:55 [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores Alexander A. Klimov
2020-07-08 10:48 ` Michal Simek
2020-07-14 23:57 ` Laurent Pinchart
2020-07-15  6:33   ` Alexander A. Klimov
2020-07-15 21:44 ` Rob Herring

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).