Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(634)

Issue 323820043: [dfvfs] Fix bug with negative seeks for encrypted, encoded, and compressed streams. #183 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 months, 2 weeks ago by dc3.plaso
Modified:
3 months, 3 weeks ago
Reviewers:
Joachim Metz, bluenlonesome83
CC:
kiddi, log2timeline-dev_googlegroups.com, onager
Visibility:
Public.

Description

[dfvfs] Fix bug with negative seeks for encrypted, encoded, and compressed streams. #183

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -6 lines) Patch
M dfvfs/file_io/compressed_stream_io.py View 1 chunk +2 lines, -2 lines 0 comments Download
M dfvfs/file_io/encoded_stream_io.py View 1 chunk +2 lines, -2 lines 0 comments Download
M dfvfs/file_io/encrypted_stream_io.py View 1 chunk +2 lines, -2 lines 0 comments Download
M tests/file_io/test_lib.py View 2 chunks +6 lines, -0 lines 0 comments Download

Messages

Total messages: 6
dc3.plaso
5 months, 2 weeks ago (2017-05-09 19:28:05 UTC) #1
Joachim Metz
Let me double check if the proposed solution does not introduce offset drift.
5 months, 2 weeks ago (2017-05-10 05:33:27 UTC) #2
BlueNLonesome83
5 months, 1 week ago (2017-05-12 11:44:44 UTC) #3
dc3.plaso
On 2017/05/10 05:33:27, Joachim Metz wrote: > Let me double check if the proposed solution ...
3 months, 4 weeks ago (2017-06-23 13:36:15 UTC) #4
Joachim Metz
Sorry for the delay, I took a closer look and LGTM Thanks for fixing this ...
3 months, 4 weeks ago (2017-06-25 10:05:34 UTC) #5
Joachim Metz
3 months, 4 weeks ago (2017-06-25 11:03:06 UTC) #6

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 80a51fa-tainted