lots of cppcheck fixes

[apt.git] / apt-pkg / contrib / netrc.cc

diff --git a/apt-pkg/contrib/netrc.cc b/apt-pkg/contrib/netrc.cc
index 18652730608de9fcc7ec40a4996a9717ed0c4886..9ff5796c53dde219d961e5884a1f3eb57bc2ac51 100644 (file)
--- a/apt-pkg/contrib/netrc.cc
+++ b/apt-pkg/contrib/netrc.cc
@@ -13,6 +13,7 @@
                                                                        /*}}}*/
 
 #include <apt-pkg/configuration.h>
                                                                        /*}}}*/
 
 #include <apt-pkg/configuration.h>

+#include <apt-pkg/fileutl.h>

 #include <iostream>
 #include <stdio.h>
 #include <stdlib.h>
 #include <iostream>
 #include <stdio.h>
 #include <stdlib.h>


@@ -46,10 +47,7 @@ int parsenetrc (char *host, char *login, char *password, char *netrcfile = NULL)
   int specific_login = (login[0] != 0);
   char *home = NULL;
   bool netrc_alloc = false;
   int specific_login = (login[0] != 0);
   char *home = NULL;
   bool netrc_alloc = false;

-  int state = NOTHING;

 
 

-  char state_login = 0;        /* Found a login keyword */
-  char state_password = 0;     /* Found a password keyword */

   int state_our_login = false;  /* With specific_login,
                                    found *our* login name */
 
   int state_our_login = false;  /* With specific_login,
                                    found *our* login name */
 


@@ -80,6 +78,10 @@ int parsenetrc (char *host, char *login, char *password, char *netrcfile = NULL)
     bool done = false;
     char netrcbuffer[256];
 
     bool done = false;
     char netrcbuffer[256];
 

+    int state = NOTHING;
+    char state_login = 0;        /* Found a login keyword */
+    char state_password = 0;     /* Found a password keyword */
+

     while (!done && fgets(netrcbuffer, sizeof (netrcbuffer), file)) {
       tok = strtok_r (netrcbuffer, " \t\n", &tok_buf);
       while (!done && tok) {
     while (!done && fgets(netrcbuffer, sizeof (netrcbuffer), file)) {
       tok = strtok_r (netrcbuffer, " \t\n", &tok_buf);
       while (!done && tok) {


@@ -99,7 +101,10 @@ int parsenetrc (char *host, char *login, char *password, char *netrcfile = NULL)
           }
           break;
         case HOSTFOUND:
           }
           break;
         case HOSTFOUND:

-          if (!strcasecmp (host, tok)) {
+          /* extended definition of a "machine" if we have a "/"
+             we match the start of the string (host.startswith(token) */
+         if ((strchr(host, '/') && strstr(host, tok) == host) ||
+             (!strcasecmp (host, tok))) {

             /* and yes, this is our host! */
             state = HOSTVALID;
             retcode = 0; /* we did find our host */
             /* and yes, this is our host! */
             state = HOSTVALID;
             retcode = 0; /* we did find our host */


@@ -148,25 +153,49 @@ int parsenetrc (char *host, char *login, char *password, char *netrcfile = NULL)
 void maybe_add_auth (URI &Uri, string NetRCFile)
 {
   if (_config->FindB("Debug::Acquire::netrc", false) == true)
 void maybe_add_auth (URI &Uri, string NetRCFile)
 {
   if (_config->FindB("Debug::Acquire::netrc", false) == true)

-     std::clog << "maybe_add_auth: " << NetRCFile << std::endl;
+     std::clog << "maybe_add_auth: " << (string)Uri 
+              << " " << NetRCFile << std::endl;

   if (Uri.Password.empty () == true || Uri.User.empty () == true)
   {
     if (NetRCFile.empty () == false)
     {
       char login[64] = "";
       char password[64] = "";
   if (Uri.Password.empty () == true || Uri.User.empty () == true)
   {
     if (NetRCFile.empty () == false)
     {
       char login[64] = "";
       char password[64] = "";

-      char *netrcfile = strdup (NetRCFile.c_str ());
-      char *host = strdup (Uri.Host.c_str ());
+      char *netrcfile = strdup(NetRCFile.c_str());

 
 

-      if (host && 0 == parsenetrc (host, login, password, netrcfile))
+      // first check for a generic host based netrc entry
+      char *host = strdup(Uri.Host.c_str());
+      if (host && parsenetrc (host, login, password, netrcfile) == 0)

       {
       {

+        if (_config->FindB("Debug::Acquire::netrc", false) == true)
+           std::clog << "host: " << host 
+                     << " user: " << login
+                     << " pass-size: " << strlen(password)
+                     << std::endl;

         Uri.User = string (login);
         Uri.Password = string (password);
         Uri.User = string (login);
         Uri.Password = string (password);

+       free(netrcfile);
+       free(host);
+       return;

       }
       }

+      free(host);

 
 

-      if (host)
-        free (host);
-      free (netrcfile);
+      // if host did not work, try Host+Path next, this will trigger
+      // a lookup uri.startswith(host) in the netrc file parser (because
+      // of the "/"
+      char *hostpath = strdup(string(Uri.Host+Uri.Path).c_str());
+      if (hostpath && parsenetrc (hostpath, login, password, netrcfile) == 0)
+      {
+        if (_config->FindB("Debug::Acquire::netrc", false) == true)
+           std::clog << "hostpath: " << hostpath
+                     << " user: " << login
+                     << " pass-size: " << strlen(password)
+                     << std::endl;
+        Uri.User = string (login);
+        Uri.Password = string (password);
+      }
+      free(netrcfile);
+      free(hostpath);

     }
   }
 }
     }
   }
 }