From patchwork Sun Feb  7 04:27:37 2021
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: asan: unwind-ia64.c: stack buffer overflow
X-Patchwork-Submitter: Alan Modra via Binutils <binutils@sourceware.org>
X-Patchwork-Id: 48636
Message-Id: <20210207042737.GO5348@bubble.grove.modra.org>
To: binutils@sourceware.org
Date: Sun, 7 Feb 2021 14:57:37 +1030
From: Alan Modra via Binutils <binutils@sourceware.org>
List-Id: Binutils mailing list <binutils.sourceware.org>

Printing "invalid" is better than printing an uninitialised buffer
and occasionally running off the end of the buffer.

	* unwind-ia64.c (unw_print_xyreg): Don't leave output buffer
	uninitialised on invalid input.


-- 
Alan Modra
Australia Development Lab, IBM

diff --git a/binutils/unwind-ia64.c b/binutils/unwind-ia64.c
index 6eef917a6a9..fb22f3fd54f 100644
--- a/binutils/unwind-ia64.c
+++ b/binutils/unwind-ia64.c
@@ -156,6 +156,10 @@ unw_print_xyreg (char *cp, unsigned int x, unsigned int ytreg)
     case 2: /* br */
       sprintf (cp, "b%u", (ytreg & 0x1f));
       break;
+
+    default:
+      strcpy (cp, "invalid");
+      break;
     }
 }