Re: pg_autovacuum/Win32 Fixes - Mailing list pgsql-patches

From Neil Conway
Subject Re: pg_autovacuum/Win32 Fixes
Date
Msg-id 41F2273D.2060502@samurai.com
Whole thread Raw
In response to pg_autovacuum/Win32 Fixes  ("Dave Page" <dpage@vale-housing.co.uk>)
Responses Re: pg_autovacuum/Win32 Fixes  (Neil Conway <neilc@samurai.com>)
List pgsql-patches
Dave Page wrote:
> Theis patch supercedes *all* my earlier ones from today - apologies for
> the noise, clearly I need a beer or 3 and a few nights away from the
> laptop.

@@ -1166,7 +1166,9 @@
          exit(0);
  #ifdef WIN32
      case 'E':
-        args->service_dependencies = optarg;
+        ZeroMemory(deps, sizeof(deps));
+        snprintf(deps, sizeof(deps) - 2, "%s", optarg);
+        args->service_dependencies = (char *)deps;
          break;
      case 'I':
          args->install_as_service++;

After googling around I can see what this code is intended to do; in the
future a comment might be nice. Also, why not strncpy()?

Barring any objections I'll apply this patch to REL8_0_STABLE and HEAD
on Monday.

-Neil

pgsql-patches by date:

Previous
From: Neil Conway
Date:
Subject: Re: regression: pass with no default WITH OIDS
Next
From: Neil Conway
Date:
Subject: vacuum hint on elog