[#89] Multiple servers in configuration

agroal · Jul 15, 2020 · cf7b98b · cf7b98b
1 parent 12340f7
commit cf7b98b
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/src/libpgagroal/configuration.c b/src/libpgagroal/configuration.c
@@ -142,7 +142,7 @@ pgagroal_read_configuration(char* filename, void* shmem)
    char* ptr = NULL;
    size_t max;
    struct configuration* config;
-   int idx_server = -1;
+   int idx_server = 0;
    struct server srv;
 
    file = fopen(filename, "r");
@@ -169,19 +169,18 @@ pgagroal_read_configuration(char* filename, void* shmem)
                memcpy(&section, line + 1, max);
                if (strcmp(section, "pgagroal"))
                {
-                  idx_server++;
-
-                  if (idx_server > 0 && idx_server < NUMBER_OF_SERVERS)
+                  if (idx_server > 0 && idx_server <= NUMBER_OF_SERVERS)
                   {
-                     memcpy(&(config->servers[idx_server]), &srv, sizeof(struct server));
+                     memcpy(&(config->servers[idx_server - 1]), &srv, sizeof(struct server));
                   }
-                  else if (idx_server >= NUMBER_OF_SERVERS)
+                  else if (idx_server > NUMBER_OF_SERVERS)
                   {
                      printf("Maximum number of servers exceeded\n");
                   }
 
                   memset(&srv, 0, sizeof(struct server));
                   memcpy(&srv.name, &section, strlen(section));
+                  idx_server++;
                }
             }
          }
@@ -679,10 +678,9 @@ pgagroal_read_configuration(char* filename, void* shmem)
       }
    }
 
-   if (idx_server != -1 && strlen(srv.name) > 0)
+   if (strlen(srv.name) > 0)
    {
-      memcpy(&(config->servers[idx_server]), &srv, sizeof(struct server));
-      idx_server++;
+      memcpy(&(config->servers[idx_server - 1]), &srv, sizeof(struct server));
    }
 
    config->number_of_servers = idx_server;