IconWindow__HookFunc_ToolbarLocationStringFunc() did
stupid things like using a not initialized FileInfoBlock
struct on the stack instead of using AllocDosObject(DOS_FIB).
It also passed &fib (instead of ACCESS_READ or similiar) as
2nd param to Lock ("Lock(str, &fib)")
memleak: In OM_DISPOSE kill notifications which were added
to global prefs obj (which goes away only when Wanderer is
quit).
MUIM_IconWindow_Remove did stupid things too, like
calling methods on _window(self) which is not even an
object but a "struct Window *".
git-svn-id: https://svn.aros.org:8080/svn/aros/trunk/AROS@26313 fb15a70f-31f2-0310-bbcc-cdcc74a49acc