bugConcurrent Versions System - Bugs: bug #14601, packetizing_buffer_input() hang...

 
 

bug #14601: packetizing_buffer_input() hang for small need

Submitter:  Serguei E Leontiev <serge3leo>
Submitted:  Tue 20 Sep 2005 07:01:44 PM UTC
Votes: 50
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  Need Info
Privacy:  Public Assigned to:  None
Open/Closed:  Open Release: 
Fixed Release:  None Fixed Feature Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 28 Oct 2005 02:40:11 AM UTC, comment #4: 

P.S.

I try to minimize regression test from  patch bug #14506 sainty.sh for `cvs –x’.

Serguei E Leontiev <serge3leo>
Fri 28 Oct 2005 02:38:24 AM UTC, comment #3: 

Sorry, I misread this.  I thought this was a bug in compression_buffer_input like bug #14840.  I'll try and take a further look at this again later.

Thanks,

Derek Robert Price <dprice>
Group administrator
Fri 28 Oct 2005 02:27:11 AM UTC, comment #2: 

This bug (packetizing_buffer_input()) detected when communicate GSS-API client – gserver.

As `unsigned int’ overflow in lines:
src/buffer.c:1403 need -= copy;

and

src/buffer.c:1554 need -= tcount;

May be correct this bug #14601 – corrected bug #14840 also. But zlib correction don’t affect to GSS-API input.

I create test case for this bug #14601 on next week.

Sorry for my best English.

Serguei E Leontiev <serge3leo>
Thu 27 Oct 2005 06:22:33 PM UTC, comment #1: 

I think this is a duplicate of bug #14840.  Serguei, can you verify that my fix there also fixes your problem?  Also, as I asked Rahul to do in that bug, could you provide a short CVS session writted in shell script which reproduces this bug?

Thanks,

Derek Robert Price <dprice>
Group administrator
Tue 20 Sep 2005 07:01:44 PM UTC, original submission:  

src/buffer.c:packetizing_buffer_input() incorrect proceesing, when agrument `need' too small.

Call it with need=1 cause hang.

Sorry for my best English
--
Seruei E. Leontiev <lse@CryptoPro.ru>

Serguei E Leontiev <serge3leo>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #3150:  cvs-1.12.12.1-buffer.patch added by serge3leo (686B - text/plain - patch to improve buffer input (SuSE 9.3 & RHEL4 - OK))

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 50 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

Only logged-in users can vote.

 

Follow 5 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2005-10-27 dprice StatusNone Need Info
2005-10-02 serge3leo Carbon-Copy- Added serge3leo
2005-10-02 serge3leo Carbon-Copy- Added -email is unavailable-
2005-09-20 serge3leo Attached File- Added cvs-1.12.12.1-buffer.patch, #2972
    Carbon-Copy- Added -email is unavailable-

Back to the top

Powered by Savane 3.13-f8d8.
Corresponding source code