Project

Profile

Help

HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. "Read more...":https://support.plan.io/news/187

Feature #747319

closed

Refactor network stream uncompressing

Added by Marko Lindqvist almost 5 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Normal
Category:
General
Sprint/Milestone:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Split from Bug #746878.

Instead of assuming that certain size buffer is big enough for uncompressed data and having an hard error if it doesn't, uncompressing should be retried with bigger buffer if needed.


Files

Actions #1

Updated by Sim Sulla almost 5 years ago

I have corrected #747319. Now we retry with a bigger buffer. There is still a max decompression to prevent a zip bomb.

See https://github.com/freeciv/freeciv/pull/10

Actions #2

Updated by Marko Lindqvist almost 5 years ago

Please read doc/CodingStyle. Use spaces instead of tabs and correct the indentation in general.

The commit messages should list also who reported the issue, and who made the patch if different from the commit author. See existing git logs for examples of commit messages.

Actions #3

Updated by Sim Sulla almost 5 years ago

I have corrected the indentation (sorry about the tabs). The correction commit follows what seems to be custom in other commits (i.e. referencing the hrm in the end of the message). The hrm provides info on who reported the issue, and the git commit about who did the correction. All is in the pull request.

Actions #4

Updated by Marko Lindqvist almost 5 years ago

Sim Sulla wrote:

The hrm provides info on who reported the issue

The main reason we've been adding them directly to commit messages is to make it easier for Release Manager to collect them for credits in PEOPLE. There's thousands of commits between major versions, so clicking and reading through all the tickets for figuring out the reporters would make already big task enormous. Also, as demonstrated by the loss of ticket trackers in the past, information in the tickets is not as permanent as in commit messages.
Reporter or patch author doesn't need to be specified if it's the person making the commit (= the default is that committer has done both of those actions)

Actions #5

Updated by Marko Lindqvist almost 5 years ago

A bit updated versions attached.

- Also S2_6 version of the patch
- Corrected indentation
- Removed outdated comment
- Made initial buffer size smaller (100 > 80) than traditionally now that it's not necessary to be big enough for all cases
Made increase smaller (100 -> 50)

Actions #6

Updated by Marko Lindqvist almost 5 years ago

  • Status changed from Resolved to Closed
  • Assignee set to Marko Lindqvist

Also available in: Atom PDF