Fix use-after-free when deleting station

Eventcmd uses both, selStation and selSong. Fixes PromyLOPh#617.
thedmd · May 17, 2017 · b732d8c · b732d8c
1 parent 21b1675
commit b732d8c
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/ui_act.c b/src/ui_act.c
@@ -228,11 +228,13 @@ BarUiActCallback(BarUiActDeleteStation) {
 				/* drain playlist */
 				PianoDestroyPlaylist (PianoListNextP (app->playlist));
 				app->playlist->head.next = NULL;
+				selSong = NULL;
 			}
 			app->nextStation = NULL;
 			/* XXX: usually we shoudn’t touch cur*, but DELETE_STATION destroys
 			 * station struct */
 			app->curStation = NULL;
+			selStation = NULL;
 		}
 		BarUiActDefaultEventcmd ("stationdelete");
 	}