GNU bug report logs - #62216
Odd file corruption in clojure mode and emacs 30 HEAD

Previous Next

Full log

Message #32 received at 62216 <at> debbugs.gnu.org (full text, mbox):

From: Jonathon McKitrick <jcm <at> SDF.ORG>
To: 62216 <at> debbugs.gnu.org
Subject: Narrowed the issue to one line
Date: Sun, 20 Aug 2023 13:38:26 +0000

Progress!

This line is the culprit, around 4807:

ptrdiff_t trytry = min (gap_size, READ_BUF_SIZE);

When I replace it with this, from an older version:

ptrdiff_t trytry = min (total - inserted, READ_BUF_SIZE);

The problem disappears.

This change was made to address bug #9800, when inserting large files from /proc.

I don't understand the code well enough to understand why this change is causing the issue I'm seeing,
or if my 'fix' will break something else. I just know it's working for me and doesn't seem to cause issues otherwise.
However, it ignores 'gap_size', so I'm pretty sure that's not the correct way to fix this code.
Maybe it should be used 2 lines below? Again, I don't have the context to do anything but guess.

I'll keep testing locally, but I'd be interested in feedback on this change.

Jonathon McKitrick
--
'My other computer is your Linux box.'

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.