From 2d63fd63c17aa6cc3ce10e6793c1fc0b4752d710 Mon Sep 17 00:00:00 2001
From: Tim Edwards <tim@opencircuitdesign.com>
Date: Sat, 4 Mar 2023 10:05:57 -0500
Subject: [PATCH] Corrected the wrong order of arguments in an strncpy()
 command that was introduced in a recent pull request, as pointed out by Mitch
 Bailey in github issue #72.

---
 VERSION        | 2 +-
 base/verilog.c | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/VERSION b/VERSION
index b588e7d..110fd1b 100644
--- a/VERSION
+++ b/VERSION
@@ -1 +1 @@
-1.5.247
+1.5.248
diff --git a/base/verilog.c b/base/verilog.c
index 07bdb9d..7aab85e 100644
--- a/base/verilog.c
+++ b/base/verilog.c
@@ -1590,11 +1590,11 @@ skip_endmodule:
 			if (i != -1)
 			    snprintf(nodename, MAX_STR_LEN, "%s[%d]", noderoot, i);
 			else
-			    strncpy(nodename, MAX_STR_LEN, lhs->name);
+			    strncpy(nodename, lhs->name, MAX_STR_LEN - 1);
 			if (j != -1)
 			    snprintf(assignname, MAX_STR_LEN, "%s[%d]", assignroot, j);
 			else
-			    strncpy(assignname, MAX_STR_LEN, rhs->name);
+			    strncpy(assignname, rhs->name, MAX_STR_LEN - 1);
 
 			join(nodename, assignname);