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, 4 insertions, 3 deletions

diff --git a/networking/httpd.c b/networking/httpd.c
index bd3a5a097..1dae602ee 100644
--- a/networking/httpd.c
+++ b/networking/httpd.c
@@ -754,10 +754,11 @@ static int parse_conf(const char *path, int flag)
                 filename = alloca(strlen(path) + sizeof(HTTPD_CONF) + 2);
                 sprintf((char *)filename, "%s/%s", path, HTTPD_CONF);
 #else
-                const char *sd = need_system_drive(path);
+                const char *sd = "";
+                if (root_len(path) == 0 && (path[0] == '/' || path[0] == '\\'))
+                        sd = get_system_drive();
 
-                filename = auto_string(xasprintf("%s%s/%s", sd ? sd : "",
-                                path, HTTPD_CONF));
+                filename = auto_string(xasprintf("%s%s/%s", sd, path, HTTPD_CONF));
 #endif
         }