- MakeTraceConcrete now yields nicer choices, e.g. "Agent1" or "Nonce2".

src/arachne.c

@@ -1706,7 +1706,7 @@ createNewTermGeneric (Termlist tl, Term t)
   Term newterm;
 
   /* Determine first free number */
-  freenumber = 0;
+  freenumber = sys->maxruns;
   tlscan = tl;
   while (tlscan != NULL)
     {
@@ -1832,6 +1832,8 @@ deleteNewTerm (Term t)
  * People find reading variables in attack outputs difficult.
  * Thus, we instantiate them in a sensible way to make things more readable.
  *
+ * This happens after sys->maxruns is fixed. Intruder constants thus are numbered from sys->maxruns onwards.
+ *
  * \sa makeTraceClass
  */
 Termlist

src/dotout.c

@@ -67,28 +67,37 @@ printVisualRun (int rid)
   int displayr;
   int display;
 
-  displayi = 0;
+  if (rid < sys->maxruns)
-  displayr = 0;
-  for (run = 0; run < rid; run++)
     {
-      if (sys->runs[run].protocol != INTRUDER)
+      // < sys->maxruns means normal thing (not from makeTraceConcrete)
+      displayi = 0;
+      displayr = 0;
+      for (run = 0; run < rid; run++)
 	{
-	  displayr++;
+	  if (sys->runs[run].protocol != INTRUDER)
+	    {
+	      displayr++;
+	    }
+	  else
+	    {
+	      displayi++;
+	    }
+	}
+      if (sys->runs[rid].protocol == INTRUDER)
+	{
+	  display = sys->maxruns + displayi + 1;
 	}
       else
 	{
-	  displayi++;
+	  display = displayr + 1;
 	}
-    }
+      eprintf ("#%i", display);
-  if (sys->runs[rid].protocol == INTRUDER)
-    {
-      display = sys->maxruns + displayi + 1;
     }
   else
     {
-      display = displayr + 1;
+      // >= sys->maxruns means intruder choice
+      eprintf ("%i", (rid - sys->maxruns + 1));
     }
-  eprintf ("#%i", display);
 }
 
 //! Remap term stuff