From eeeaf49ec7d43908308b52d7b73b04e77879a860 Mon Sep 17 00:00:00 2001 From: Tim Edwards Date: Fri, 8 Mar 2024 09:33:13 -0500 Subject: [PATCH] Corrected an error in defGetType() that incorrectly looks at a lefLayer's "via" record even when the layer might not be a via, causing potential issues with uninitialized variables. Not sure if this is related to the bug that started this investigation, but it was the only thing that looked relevant. --- VERSION | 2 +- lef/defWrite.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/VERSION b/VERSION index 996d9654..70773ddf 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -8.3.463 +8.3.464 diff --git a/lef/defWrite.c b/lef/defWrite.c index 4b8c742f..907f9acb 100644 --- a/lef/defWrite.c +++ b/lef/defWrite.c @@ -2040,7 +2040,7 @@ defGetType(ttype, lefptr, do_vias) { lefl = (lefLayer *)HashGetValue(he); if (lefl && (do_vias == FALSE) && (contact == CLASS_VIA) && - (lefl->info.via.lr != NULL)) + (lefl->lefClass == CLASS_VIA)) continue; /* Skip VIA definitions if do_vias is FALSE */ if (lefl && ((contact == lefl->lefClass) ||