luke
/
magic
mirror of https://github.com/RTimothyEdwards/magic.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Corrected an error that allows a cell to be renamed with the name 

of an existing cell, which does not appear to be a fatal problem,
but must be causing a memory leak.  Also:  Corrected an error in
extresist when reading .sim files, in code that is slated to be
overhauled, so this is just a patch to avoid a crash condition.
This commit is contained in:
R. Timothy Edwards 2025-10-14 10:36:07 -04:00
parent c42db8e71b
commit 0e84616af8
3 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
VERSION
View File

@ -1 +1 @@
8.3.561 8.3.562

8
database/DBcellname.c
View File

@ -117,6 +117,14 @@ DBCellRename(cellname, newname, doforce)
return FALSE; return FALSE;
} }
/* Cannot rename a cell with the name of an existing cell */
entry = HashLookOnly(&dbCellDefTable, newname);
if (entry != NULL)
{
TxError("Cannot rename; cell \"%s\" already exists!\n", newname);
return FALSE;
}
/* Disallow renaming if the cell has the READONLY flag set, */ /* Disallow renaming if the cell has the READONLY flag set, */
/* because the cellname must match the name in the GDS */ /* because the cellname must match the name in the GDS */
/* file referenced. */ /* file referenced. */

5
resis/ResReadSim.c
View File

@ -198,9 +198,10 @@ ResReadSim(simfile, fetproc, capproc, resproc, attrproc, mergeproc, subproc)
result = 1; result = 1;
break; break;
} }
if (fettype == -1) if (fettype < 0)
{ {
TxError("Error in Reading device line of sim file.\n"); TxError("Error in Reading device line of sim file:");
TxError("Amgiguous or unknown device.\n");
result = 1; result = 1;
} }
else if (fettype == DBNumTypes) else if (fettype == DBNumTypes)