win32: code shrink system drive handling

A previous commit (e3bfe3695) revised the use of getsysdir() to
obtain the system directory, and hence the system drive.  See the
commit message for the history to that point.

Further improvements are possible:

- Remove getsysdir() and push the calls to GetSystemDirectory()
  down into get_system_drive() and get_proc_addr().

- Check the return value of GetSystemDirectory().  It's unlikely
  to fail, but better safe than sorry.

- Instead of making all callers of get_system_drive() check for a
  NULL return value always return a non-NULL pointer.  If the drive
  can't be found an empty string is returned instead (which is what
  the callers were using anyway).

- The function need_system_drive() was only used in one place (in
  httpd).  Move the code there and remove the function.

- Use concat_path_file() where possible.

Saves 76-144 bytes.

1 files changed, 3 insertions, 3 deletions

diff --git a/libbb/appletlib.c b/libbb/appletlib.c
index 54be59f2d..97dfb3df8 100644
--- a/libbb/appletlib.c
+++ b/libbb/appletlib.c
@@ -781,7 +781,7 @@ static void install_links(const char *busybox, int use_symbolic_links,
         unsigned i;
         int rc;
 #  if ENABLE_PLATFORM_MINGW32
-        const char *sd = NULL;
+        const char *sd = "";
 
         if (custom_install_dir != NULL) {
                 bb_make_directory(custom_install_dir, 0755, FILEUTILS_RECUR);
@@ -789,7 +789,7 @@ static void install_links(const char *busybox, int use_symbolic_links,
         else {
                 sd = get_system_drive();
                 for (i=1; i<ARRAY_SIZE(install_dir); ++i) {
-                        fpc = xasprintf("%s%s", sd ?: "", install_dir[i]);
+                        fpc = concat_path_file(sd, install_dir[i]);
                         bb_make_directory(fpc, 0755, FILEUTILS_RECUR);
                         free(fpc);
                 }
@@ -802,7 +802,7 @@ static void install_links(const char *busybox, int use_symbolic_links,
 
         for (i = 0; i < ARRAY_SIZE(applet_main); i++) {
 #  if ENABLE_PLATFORM_MINGW32
-                fpc = xasprintf("%s%s/%s.exe", sd ?: "",
+                fpc = xasprintf("%s%s/%s.exe", sd,
                                 custom_install_dir ?: install_dir[APPLET_INSTALL_LOC(i)],
                                 appname);
 #  else