diff for duplicates of <1362555258.23932.573.camel@linux-s257.site> diff --git a/a/1.txt b/N1/1.txt index 67368e4..2f5d8fe 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -4,7 +4,7 @@ Hi Matt, > 於 五,2013-03-01 於 16:31 +0000,Matt Fleming 提到: > > On Fri, 2013-03-01 at 15:17 +0000, Matt Fleming wrote: > > > On Fri, 2013-03-01 at 11:20 +0800, Lee, Chun-Yi wrote: -> > > > From: Michael Schroeder <mls-IBi9RG/b67k@public.gmane.org> +> > > > From: Michael Schroeder <mls@suse.com> > > > > > > > > On HP z220 system (firmware version 1.54), some EFI variables are incorrectly > > > > named : @@ -43,7 +43,7 @@ Hi Matt, Frederic confirmed your patch works to him for fix issue on HP machine. -Tested-by: Frederic Crozat <fcrozat-IBi9RG/b67k@public.gmane.org> +Tested-by: Frederic Crozat <fcrozat@suse.com> But, I suggest we just use the length with NULL in variable_name but not VariableNameSize that was returned by GetNextVariableName. @@ -55,7 +55,7 @@ My thinking is following... > > --- > > > > >From 4b2ef72bca72039717efe4570ec858a86d565b34 Mon Sep 17 00:00:00 2001 -> > From: Matt Fleming <matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> +> > From: Matt Fleming <matt.fleming@intel.com> > > Date: Fri, 1 Mar 2013 14:49:12 +0000 > > Subject: [PATCH] efivars: Sanitise string length returned by > > GetNextVariableName() @@ -76,13 +76,13 @@ My thinking is following... > > Since 'variable_name' is a string, we can validate its size by > > searching for the terminating NULL character. > > -> > Reported-by: Frederic Crozat <fcrozat-IBi9RG/b67k@public.gmane.org> -> > Cc: Matthew Garrett <mjg59-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org> -> > Cc: Josh Boyer <jwboyer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> -> > Cc: Michael Schroeder <mls-IBi9RG/b67k@public.gmane.org> -> > Cc: Lee, Chun-Yi <jlee-IBi9RG/b67k@public.gmane.org> -> > Cc: Lingzhu Xiang <lxiang-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> -> > Signed-off-by: Matt Fleming <matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> +> > Reported-by: Frederic Crozat <fcrozat@suse.com> +> > Cc: Matthew Garrett <mjg59@srcf.ucam.org> +> > Cc: Josh Boyer <jwboyer@redhat.com> +> > Cc: Michael Schroeder <mls@suse.com> +> > Cc: Lee, Chun-Yi <jlee@suse.com> +> > Cc: Lingzhu Xiang <lxiang@redhat.com> +> > Signed-off-by: Matt Fleming <matt.fleming@intel.com> > > --- > > drivers/firmware/efivars.c | 30 ++++++++++++++++++++++++++++++ > > 1 file changed, 30 insertions(+) diff --git a/a/content_digest b/N1/content_digest index 8fa8413..b1983cf 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,18 +2,17 @@ "ref\01362151068.2842.440.camel@mfleming-mobl1.ger.corp.intel.com\0" "ref\01362155493.2842.446.camel@mfleming-mobl1.ger.corp.intel.com\0" "ref\01362181299.23932.168.camel@linux-s257.site\0" - "ref\01362181299.23932.168.camel-ONCj+Eqt86TasUa73XJKwA@public.gmane.org\0" - "From\0joeyli <jlee-IBi9RG/b67k@public.gmane.org>\0" + "From\0joeyli <jlee@suse.com>\0" "Subject\0Re: [PATCH] efivarfs: fix abnormal GUID in variable name by using strcpy to replace null with dash\0" "Date\0Wed, 06 Mar 2013 15:34:18 +0800\0" - "To\0Matt Fleming <matt-HNK1S37rvNbeXh+fF434Mdi2O/JbrIOy@public.gmane.org>\0" - "Cc\0linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" - linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - Michael Schroeder <mls-IBi9RG/b67k@public.gmane.org> - Josh Boyer <jwboyer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> - Peter Jones <pjones-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> - Matthew Garrett <mjg59-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org> - " Frederic Crozat <fcrozat-IBi9RG/b67k@public.gmane.org>\0" + "To\0Matt Fleming <matt@console-pimps.org>\0" + "Cc\0linux-efi@vger.kernel.org" + linux-kernel@vger.kernel.org + Michael Schroeder <mls@suse.com> + Josh Boyer <jwboyer@redhat.com> + Peter Jones <pjones@redhat.com> + Matthew Garrett <mjg59@srcf.ucam.org> + " Frederic Crozat <fcrozat@suse.com>\0" "\00:1\0" "b\0" "Hi Matt, \n" @@ -22,7 +21,7 @@ "> \346\226\274 \344\272\224\357\274\2142013-03-01 \346\226\274 16:31 +0000\357\274\214Matt Fleming \346\217\220\345\210\260\357\274\232\n" "> > On Fri, 2013-03-01 at 15:17 +0000, Matt Fleming wrote:\n" "> > > On Fri, 2013-03-01 at 11:20 +0800, Lee, Chun-Yi wrote:\n" - "> > > > From: Michael Schroeder <mls-IBi9RG/b67k@public.gmane.org>\n" + "> > > > From: Michael Schroeder <mls@suse.com>\n" "> > > > \n" "> > > > On HP z220 system (firmware version 1.54), some EFI variables are incorrectly\n" "> > > > named :\n" @@ -61,7 +60,7 @@ "\n" "Frederic confirmed your patch works to him for fix issue on HP machine.\n" "\n" - "Tested-by: Frederic Crozat <fcrozat-IBi9RG/b67k@public.gmane.org>\n" + "Tested-by: Frederic Crozat <fcrozat@suse.com>\n" "\n" "But, I suggest we just use the length with NULL in variable_name but not\n" "VariableNameSize that was returned by GetNextVariableName.\n" @@ -73,7 +72,7 @@ "> > ---\n" "> > \n" "> > >From 4b2ef72bca72039717efe4570ec858a86d565b34 Mon Sep 17 00:00:00 2001\n" - "> > From: Matt Fleming <matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>\n" + "> > From: Matt Fleming <matt.fleming@intel.com>\n" "> > Date: Fri, 1 Mar 2013 14:49:12 +0000\n" "> > Subject: [PATCH] efivars: Sanitise string length returned by\n" "> > GetNextVariableName()\n" @@ -94,13 +93,13 @@ "> > Since 'variable_name' is a string, we can validate its size by\n" "> > searching for the terminating NULL character.\n" "> > \n" - "> > Reported-by: Frederic Crozat <fcrozat-IBi9RG/b67k@public.gmane.org>\n" - "> > Cc: Matthew Garrett <mjg59-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org>\n" - "> > Cc: Josh Boyer <jwboyer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>\n" - "> > Cc: Michael Schroeder <mls-IBi9RG/b67k@public.gmane.org>\n" - "> > Cc: Lee, Chun-Yi <jlee-IBi9RG/b67k@public.gmane.org>\n" - "> > Cc: Lingzhu Xiang <lxiang-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>\n" - "> > Signed-off-by: Matt Fleming <matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>\n" + "> > Reported-by: Frederic Crozat <fcrozat@suse.com>\n" + "> > Cc: Matthew Garrett <mjg59@srcf.ucam.org>\n" + "> > Cc: Josh Boyer <jwboyer@redhat.com>\n" + "> > Cc: Michael Schroeder <mls@suse.com>\n" + "> > Cc: Lee, Chun-Yi <jlee@suse.com>\n" + "> > Cc: Lingzhu Xiang <lxiang@redhat.com>\n" + "> > Signed-off-by: Matt Fleming <matt.fleming@intel.com>\n" "> > ---\n" "> > drivers/firmware/efivars.c | 30 ++++++++++++++++++++++++++++++\n" "> > 1 file changed, 30 insertions(+)\n" @@ -223,4 +222,4 @@ " \t\t\t\t\t\t &vendor_guid);\n" " \t\t\tbreak;" -9613ecce8c10e37e3e543de451b1eb3b9adf05cbd184cbe3ce8b72f524169c20 +c55e80a5554dd3f9c047b9b5a92ef08bda49e22297a45c97688ee6b03813d0c7
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.