Patches are much more likely to be accepted if they include tests. It's up to the TDS driver maintainers.<br><br><div class="gmail_quote">On Wed, May 2, 2012 at 8:26 AM, Rob Wilkens <span dir="ltr"><<a href="mailto:robwilkens@gmail.com" target="_blank">robwilkens@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    [resending reply via reply to list only, left off list in previous
    reply]<div class="im"><br>
    <br>
    I already did the pull request on github -- and i'm not sure how to
    include the tests (I only commented that i ran the tests on github
    in the pull comments).  Or do you mean i have to somehow generate my
    own tests, which would be difficult because this requires, i think,
    a functional microsoft sql server for it to have access to to
    reproduce the error.
    <br>
    <br>
    -Rob
    <br>
    <br>
    On 05/02/2012 07:21 AM, Rodrigo Kumpera wrote:
    </div><div><div class="h5"><blockquote type="cite">Please make it a pull request on github with tests
      included.
      <div><br>
      </div>
      <div><br>
        <br>
        <div class="gmail_quote">On Tue, May 1, 2012 at 12:55 PM, Rob
          Wilkens <span dir="ltr"><<a href="mailto:robwilkens@gmail.com" target="_blank">robwilkens@gmail.com</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
            <div bgcolor="#FFFFFF" text="#000000"> <br>
              Ok, I looked up the 'selfish' way to submit a patch from
              the contributing section, and i think the below suffices
              as a patch, does anyone disagree?<br>
              <br>
              I am referring to : <a href="http://www.mono-project.com/Contributing" target="_blank">http://www.mono-project.com/Contributing</a>
              towards the bottom of the page where it simply says to
              submit the patch to the mailing list.<br>
              <br>
              Does the below suffice as a "patch" or should i figure out
              the 'github' way which i thought i saw elsewhere.<br>
              <br>
              -Rob<br>
              <br>
              On 05/01/2012 11:49 AM, Rob Wilkens wrote:
              <blockquote type="cite">
                <pre>I found out the fix for the error i reported with multiple invalid
login attempts...  It's very simple...

Mono.Data.TdsClient.TdsConnectionPool.cs

In the above file, in GetConnection(), either before:

goto retry

or after the initial

retry:

(either place should be fine)

result needs to be set to null -- that is:

result=null;

(in my testing, i put it before goto retry)

Otherwise, it keeps retrying because result has never been reset
before the while loop you are trying to enter.

I guess my next step is figuring out how to navigate github so i can submit it.

Sorry for being so public and crowding the mailing list with this.  If
someone else wants to volunteer to submit the patch for me, please
speak up.  I'll otherwise figure it out.

-Rob
</pre>
              </blockquote>
              <br>
            </div>
            <br>
            _______________________________________________<br>
            Mono-devel-list mailing list<br>
            <a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.ximian.com</a><br>
            <a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
            <br>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
  </div></div></div>

<br>_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
<br></blockquote></div><br>