public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* MdeModulePkg: Fix SetMem parameter in OnigurumaUefiPort
@ 2020-12-10  0:21 Anbazhagan, Baraneedharan
  2020-12-10  1:28 ` 回复: [edk2-devel] " gaoliming
       [not found] ` <164F36ADB5F781C1.18718@groups.io>
  0 siblings, 2 replies; 3+ messages in thread
From: Anbazhagan, Baraneedharan @ 2020-12-10  0:21 UTC (permalink / raw)
  To: devel@edk2.groups.io
  Cc: gaoliming@byosoft.com.cn, jian.j.wang@intel.com,
	hao.a.wu@intel.com

[-- Attachment #1: Type: text/plain, Size: 918 bytes --]

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3085

Coding error in converting memset call to SetMem - Length and Value
is not swapped on calling SetMem

Signed-off-by: Baraneedharan Anbazhagan <anbazhagan@hp.com>
---
MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c b/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c
index 2b2b0d420d..9aa7b0a68e 100644
--- a/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c
+++ b/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c
@@ -93,6 +93,6 @@ void* memcpy (void *dest, const void *src, unsigned int count)
 void* memset (void *dest, char ch, unsigned int count)
{
-  return SetMem (dest, ch, count);
+  return SetMem (dest, count, ch);
}
--
2.29.2.windows.3


[-- Attachment #2: Type: text/html, Size: 3271 bytes --]

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

* 回复: [edk2-devel] MdeModulePkg: Fix SetMem parameter in OnigurumaUefiPort
  2020-12-10  0:21 MdeModulePkg: Fix SetMem parameter in OnigurumaUefiPort Anbazhagan, Baraneedharan
@ 2020-12-10  1:28 ` gaoliming
       [not found] ` <164F36ADB5F781C1.18718@groups.io>
  1 sibling, 0 replies; 3+ messages in thread
From: gaoliming @ 2020-12-10  1:28 UTC (permalink / raw)
  To: devel, anbazhagan; +Cc: jian.j.wang, hao.a.wu

[-- Attachment #1: Type: text/plain, Size: 1440 bytes --]

Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

 

发件人: bounce+27952+68609+4905953+8761045@groups.io
<bounce+27952+68609+4905953+8761045@groups.io> 代表 Anbazhagan,
Baraneedharan via groups.io
发送时间: 2020年12月10日 8:21
收件人: devel@edk2.groups.io
抄送: gaoliming@byosoft.com.cn; jian.j.wang@intel.com; hao.a.wu@intel.com
主题: [edk2-devel] MdeModulePkg: Fix SetMem parameter in OnigurumaUefiPort

 

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3085

 

Coding error in converting memset call to SetMem - Length and Value

is not swapped on calling SetMem

 

Signed-off-by: Baraneedharan Anbazhagan <anbazhagan@hp.com
<mailto:anbazhagan@hp.com> >

---

MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c | 2 +-

1 file changed, 1 insertion(+), 1 deletion(-)

 

diff --git a/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c
b/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c

index 2b2b0d420d..9aa7b0a68e 100644

--- a/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c

+++ b/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c

@@ -93,6 +93,6 @@ void* memcpy (void *dest, const void *src, unsigned int
count)

 

 void* memset (void *dest, char ch, unsigned int count)

{

-  return SetMem (dest, ch, count);

+  return SetMem (dest, count, ch);

}

 

-- 

2.29.2.windows.3

 




[-- Attachment #2: Type: text/html, Size: 5787 bytes --]

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

* 回复: [edk2-devel] MdeModulePkg: Fix SetMem parameter in OnigurumaUefiPort
       [not found] ` <164F36ADB5F781C1.18718@groups.io>
@ 2020-12-14  2:00   ` gaoliming
  0 siblings, 0 replies; 3+ messages in thread
From: gaoliming @ 2020-12-14  2:00 UTC (permalink / raw)
  To: devel, gaoliming, anbazhagan; +Cc: jian.j.wang, hao.a.wu

[-- Attachment #1: Type: text/plain, Size: 2135 bytes --]

Create PR https://github.com/tianocore/edk2/pull/1219

 

Thanks

Liming

发件人: bounce+27952+68612+4905953+8761045@groups.io
<bounce+27952+68612+4905953+8761045@groups.io> 代表 gaoliming
发送时间: 2020年12月10日 9:28
收件人: devel@edk2.groups.io; anbazhagan@hp.com
抄送: jian.j.wang@intel.com; hao.a.wu@intel.com
主题: 回复: [edk2-devel] MdeModulePkg: Fix SetMem parameter in
OnigurumaUefiPort

 

Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.
com.cn> >

 

发件人: bounce+27952+68609+4905953+8761045@groups.io
<mailto:bounce+27952+68609+4905953+8761045@groups.io>
<bounce+27952+68609+4905953+8761045@groups.io
<mailto:bounce+27952+68609+4905953+8761045@groups.io> > 代表 Anbazhagan,
Baraneedharan via groups.io
发送时间: 2020年12月10日 8:21
收件人: devel@edk2.groups.io <mailto:devel@edk2.groups.io> 
抄送: gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> ;
jian.j.wang@intel.com <mailto:jian.j.wang@intel.com> ; hao.a.wu@intel.com
<mailto:hao.a.wu@intel.com> 
主题: [edk2-devel] MdeModulePkg: Fix SetMem parameter in OnigurumaUefiPort

 

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3085

 

Coding error in converting memset call to SetMem - Length and Value

is not swapped on calling SetMem

 

Signed-off-by: Baraneedharan Anbazhagan <anbazhagan@hp.com
<mailto:anbazhagan@hp.com> >

---

MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c | 2 +-

1 file changed, 1 insertion(+), 1 deletion(-)

 

diff --git a/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c
b/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c

index 2b2b0d420d..9aa7b0a68e 100644

--- a/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c

+++ b/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c

@@ -93,6 +93,6 @@ void* memcpy (void *dest, const void *src, unsigned int
count)

 

 void* memset (void *dest, char ch, unsigned int count)

{

-  return SetMem (dest, ch, count);

+  return SetMem (dest, count, ch);

}

 

-- 

2.29.2.windows.3

 




[-- Attachment #2: Type: text/html, Size: 7918 bytes --]

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

end of thread, other threads:[~2020-12-14  2:00 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-12-10  0:21 MdeModulePkg: Fix SetMem parameter in OnigurumaUefiPort Anbazhagan, Baraneedharan
2020-12-10  1:28 ` 回复: [edk2-devel] " gaoliming
     [not found] ` <164F36ADB5F781C1.18718@groups.io>
2020-12-14  2:00   ` gaoliming

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox