sfrestore: partial fix for frame allocation/deallocation

There is still a problem with allocation of screens vs frames.  A screen
can have multiple frames (for now this triggers a segfault).

1 files changed, 4 insertions, 2 deletions

diff --git a/src/actions.c b/src/actions.c
index d59cafe..73bc282 100644
--- a/src/actions.c
+++ b/src/actions.c
@@ -5662,7 +5662,7 @@ cmd_sfrestore (int interactively UNUSED, struct cmdarg **args)
       ptr++;
 
       snum = string_to_positive_int (ptr);
-      screen = screen_number(snum);
+      screen = screen_number (snum);
       if (screen)
         {
           /* clobber screen number here, frestore() doesn't need it */
@@ -5698,9 +5698,11 @@ cmd_sfrestore (int interactively UNUSED, struct cmdarg **args)
       sf_data[i].ret_restore = ret->success;
 
       cmdret_free (ret);
-      sbuf_free (cur_frames);
     }
 
+  for (i = 0; i < s_count; i++)
+    sbuf_free (sf_data[i].frames);
+
   for (i = 0; i < sf_index; i++)
     {
       int ret;