[PATCH v2] IOMUX: Fix buffer overflow in iomux_replace_device()

[Yuichiro Goto]

Use of strcat() against an uninitialized buffer would lead
to buffer overflow. This patch fixes it.

Signed-off-by: Yuichiro Goto <g...@k-tech.co.jp>
Cc: Peter Robinson <pbrobin...@gmail.com>
Cc: Andy Shevchenko <andriy.shevche...@linux.intel.com>
Cc: Nicolas Saenz Julienne <nsaenzjulie...@suse.de>
---

Changes for v2:
- Add "IOMUX" in title

 common/iomux.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/common/iomux.c b/common/iomux.c
index b9088aa3b5..27baca6931 100644
--- a/common/iomux.c
+++ b/common/iomux.c
@@ -158,7 +158,10 @@ int iomux_replace_device(const int console, const char 
*old, const char *new)
                        return -ENOMEM;
                }
 
-               strcat(tmp, ",");
+               if (!arg)
+                       strcpy(tmp, ",");
+               else
+                       strcat(tmp, ",");
                strcat(tmp, name);
 
                arg = tmp;
-- 
2.17.1