Ticket #3927 defect closed duplicate

Opened 4 years ago

Last modified 2 years ago

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

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

Change History

Changed 4 years ago by djfroofy

1

Changed 4 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?

2

Changed 2 years ago by exarkun

  • status changed from new to closed
  • resolution set to duplicate

This looks like a duplicate of #1589.

Note: See TracTickets for help on using tickets.