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, 14 insertions, 27 deletions

diff --git a/win32/mingw.c b/win32/mingw.c
index 7e42e4434..6544d11ad 100644
--- a/win32/mingw.c
+++ b/win32/mingw.c
@@ -1146,17 +1146,6 @@ static char *gethomedir(void)
         return buf;
 }
 
-static char *getsysdir(void)
-{
-        static char *buf = NULL;
-
-        if (!buf) {
-                buf = xzalloc(PATH_MAX);
-                GetSystemDirectory(buf, PATH_MAX);
-        }
-        return buf;
-}
-
 #define NAME_LEN 100
 char *get_user_name(void)
 {
@@ -2297,27 +2286,21 @@ int root_len(const char *path)
 
 const char *get_system_drive(void)
 {
-        static char *drive = NULL;
+        static const char *drive = NULL;
+        char sysdir[PATH_MAX];
         int len;
 
         if (drive == NULL) {
-                const char *sysdir = getsysdir();
-                if ((len=root_len(sysdir))) {
+                UINT ret = GetSystemDirectory(sysdir, PATH_MAX);
+                if ((ret != 0 && ret < PATH_MAX) && (len=root_len(sysdir)))
                         drive = xstrndup(sysdir, len);
-                }
+                else
+                        drive = "";
         }
 
         return getenv(BB_SYSTEMROOT) ?: drive;
 }
 
-/* Return pointer to system drive if path is of form '/file', else NULL */
-const char *need_system_drive(const char *path)
-{
-        if (root_len(path) == 0 && (path[0] == '/' || path[0] == '\\'))
-                return get_system_drive();
-        return NULL;
-}
-
 int chdir_system_drive(void)
 {
         const char *sd = get_system_drive();
@@ -2409,10 +2392,14 @@ void *get_proc_addr(const char *dll, const char *function,
                  * on Windows 7.  If it does, retry using LoadLibrary with an
                  * explicit, backslash-separated path. */
                 if (!hnd) {
-                        char *path = concat_path_file(getsysdir(), dll);
-                        slash_to_bs(path);
-                        hnd = LoadLibrary(path);
-                        free(path);
+                        char buf[PATH_MAX];
+                        UINT ret = GetSystemDirectory(buf, PATH_MAX);
+                        if (ret != 0 && ret < PATH_MAX) {
+                                char *path = concat_path_file(buf, dll);
+                                slash_to_bs(path);
+                                hnd = LoadLibrary(path);
+                                free(path);
+                        }
                 }
 
                 if (hnd)