Opened 5 years ago

Closed 4 years ago

#3927 defect closed duplicate (duplicate)

twisted.conch.scripts.cftp chunk copying can result in "RuntimeError: maximum recursion depth exceeded"

Reported by: djfroofy Owned by: djfroofy
Priority: normal Milestone:
Component: conch Keywords: cftp ssh conch
Cc: thijs Branch:
Author: Launchpad Bug:

Description

Unless buffersize' and a request' arguments are carefully adjusted, "maximum recursion depth exceeded" can occur inside of StdioClient._cbPutWrite when copying large files (~155MB if requests=10 and buffersize=48KB, for example). Please see attached patch.

Attachments (1)

cftp.py.diff (1.4 KB) - added by djfroofy 5 years ago.

Download all attachments as: .zip

Change History (3)

Changed 5 years ago by djfroofy

comment:1 Changed 5 years ago by thijs

  • Cc thijs added
  • Owner changed from z3p to djfroofy

Hi djroofy, thanks for the patch. The TwistedDevelopment policy requires unit tests for any changes, could you supply those as well?

comment:2 Changed 4 years ago by exarkun

  • Resolution set to duplicate
  • Status changed from new to closed

This looks like a duplicate of #1589.

Note: See TracTickets for help on using tickets.