Hi,

While trying to find out why TortoiseSVN/Subversion couldn't connect to a repository located on a windows domain anymore if the client wasn't part of the same domain, I stumbled upon a bug in neon. The attached patch fixes this. One patch is against trunk, the other against the 0.25.4 tag.

I've tested the patch with the help of some TortoiseSVN users who provided me with access to their repositories.

Stefan

(ccing the Subversion dev list, because they might want to include that patch for their windows binaries build).

--
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.tigris.org
Index: src/ne_sspi.c
===================================================================
--- src/ne_sspi.c       (Revision 831)
+++ src/ne_sspi.c       (Arbeitskopie)
@@ -487,7 +487,7 @@
                                       &outBufferDesc);
         freeBuffer(&inBufferDesc);
     } else {
-        if (sspiContext->continueNeeded) {
+        if (!sspiContext->continueNeeded) {
             NE_DEBUG(NE_DBG_HTTPAUTH, "sspi: Expected a token from server.\n");
             return -1;
         }
[[[
Fix endless SSPI authentication.
* src/ne_sspi.c:
  Continue authenticating only if requested.
]]]
Index: src/ne_sspi.c
===================================================================
--- src/ne_sspi.c       (Revision 831)
+++ src/ne_sspi.c       (Arbeitskopie)
@@ -487,7 +487,7 @@
                                       &outBufferDesc);
         freeBuffer(&inBufferDesc);
     } else {
-        if (sspiContext->continueNeeded) {
+        if (!sspiContext->continueNeeded) {
             NE_DEBUG(NE_DBG_HTTPAUTH, "sspi: Expected a token from server.\n");
             return -1;
         }
_______________________________________________
neon mailing list
[email protected]
http://mailman.webdav.org/mailman/listinfo/neon

Reply via email to