From b9af52108f7dce71dbcc68dfb4bfb699eb195286 Mon Sep 17 00:00:00 2001
From: h_vogt <h_vogt>
Date: Wed, 15 Jun 2016 21:09:53 +0200
Subject: [PATCH] spiceif.c, fixme marker

---
 src/frontend/spiceif.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/frontend/spiceif.c b/src/frontend/spiceif.c
index 550559e53..9bdb96d99 100644
--- a/src/frontend/spiceif.c
+++ b/src/frontend/spiceif.c
@@ -962,6 +962,9 @@ parmtovar(IFvalue *pv, IFparm *opt)
 {
     /* It is not clear whether we want to name the variable
      *   by `keyword' or by `description' */
+    /* FIXME to remove a memory leak, we have to free pv.
+       Are we shure that pv was everywhere generated by a TMALLOC
+       (like in vsrcask.c:53)? Is it used elsewhere ?  What about opt? */
 
     switch (opt->dataType & IF_VARTYPES) {
     case IF_INTEGER: