summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
bb7a36e)
As there is no need to keep autostart commands in memory, we can safely
remove them as soon as they have been executed. As we previously didn't
clear them at all during config reloads, this also fixes a memory leak.
Note that neither autostarts nor autostarts_always is executed during
config reloads, so removing them from memory is fine as an i3 restart
will cause them to be parsed again.
fixes #2044
/* Autostarting exec-lines */
if (autostart) {
/* Autostarting exec-lines */
if (autostart) {
- struct Autostart *exec;
- TAILQ_FOREACH(exec, &autostarts, autostarts) {
+ while (!TAILQ_EMPTY(&autostarts)) {
+ struct Autostart *exec = TAILQ_FIRST(&autostarts);
+
LOG("auto-starting %s\n", exec->command);
start_application(exec->command, exec->no_startup_id);
LOG("auto-starting %s\n", exec->command);
start_application(exec->command, exec->no_startup_id);
+
+ FREE(exec->command);
+ TAILQ_REMOVE(&autostarts, exec, autostarts);
+ FREE(exec);
}
}
/* Autostarting exec_always-lines */
}
}
/* Autostarting exec_always-lines */
- struct Autostart *exec_always;
- TAILQ_FOREACH(exec_always, &autostarts_always, autostarts_always) {
+ while (!TAILQ_EMPTY(&autostarts_always)) {
+ struct Autostart *exec_always = TAILQ_FIRST(&autostarts_always);
+
LOG("auto-starting (always!) %s\n", exec_always->command);
start_application(exec_always->command, exec_always->no_startup_id);
LOG("auto-starting (always!) %s\n", exec_always->command);
start_application(exec_always->command, exec_always->no_startup_id);
+
+ FREE(exec_always->command);
+ TAILQ_REMOVE(&autostarts_always, exec_always, autostarts_always);
+ FREE(exec_always);
}
/* Start i3bar processes for all configured bars */
}
/* Start i3bar processes for all configured bars */