[bug #14840] zlib.c implementation in 1.12.12+ still broken

[Rahul Bhargava]

Follow-up Comment #7, bug #14840 (project cvs):

Hello Derek -

It works!

Your fix is right on. When I looked at the 1.12.13 zlib.c code that line
looked suspicious. My testcase (add.c from cvs src tree) is working fine with
your fix. Tried all the zlib compression levels (0-9)and saw no problem with
it. 

Regarding testcase script - 

As I noted in the bug report, I see this problem from a Java client. To test
this in a scripted fashion, you could download WANdisco CVS replicator for 1
node and run the normal command "cvs -z6 ci -m change add.c" via it.  That's
the closest you can come to scripting it. We will be happy to provide you
with a license. Other not so manual ways would be to download CVS clients
like SmartCVS (foundation version is free) or JetBrains' IdeaJ IDE with CVS
plugin. We internally have our stress tests that will catch issues like
these, but we test only after a release from CVS project is out. So it will
be good to have a testcase pre-release. Let me know if we can help with
setting up a test scenario.

Best Regards,
Rahul Bhargava,



    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?func=detailitem&item_id=14840>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/