This patch changes the package dependencies and isolates the demo data file to a separate Solaris package. Please apply.. it touches only some files under trunk/misc/setup.pkg (Solaris only stuff).
LD_LIBRARY_PATH=${IOQ3_DIR}:${LD_LIBRARY_PATH} will open a security hole if LD_LIBRARY_PATH was unset before (loads libs from current dir then). Don't do that. Why don't you use the same start script we use on Linux anyways?
Hi luwdig,
I back-ported the safer routines from the Linux startup script, thanks for catching that.
The Linux and Solaris startup scripts are quite different because I need to force certain parameters on Solaris in order to make sure the game runs. If we were to share a common script, then it'd be harder to maintain (IMHO) and I don't want to impact Linux platforms with a set of Solaris changes.
Here's a modified diff with the newer scripts.
Thanks,
Vincent
Created attachment 2055 [details] Diffs for trunk/misc/setup/pkg files...LD_LIBRARY_PATH=${IOQ3_DIR}:${LD_LIBRARY_PATH} will open a security hole if LD_LIBRARY_PATH was unset before (loads libs from current dir then). Don't do that. Why don't you use the same start script we use on Linux anyways?Created attachment 2061 [details] Modified patch for safer LD_LIBRARY_PATH handling.Created attachment 2065 [details] Latest modified patch (plese use this one). Thank you.