After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 697946 - Absolutely full NTFS reported as partition error
Absolutely full NTFS reported as partition error
Status: RESOLVED FIXED
Product: gparted
Classification: Other
Component: application
GIT HEAD
Other Linux
: Normal minor
: ---
Assigned To: gparted maintainers alias
gparted maintainers alias
Depends on:
Blocks:
 
 
Reported: 2013-04-13 11:44 UTC by Mike Fleetwood
Modified: 2013-04-24 16:29 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Full ntfs fix (v1) (2.33 KB, patch)
2013-04-13 12:15 UTC, Mike Fleetwood 		none Details | Review
Full ntfs fix (v2) (2.32 KB, patch)
2013-04-14 14:11 UTC, Mike Fleetwood 		none Details | Review

Description Mike Fleetwood 2013-04-13 11:44:21 UTC 
Hi Curtis,

Found this bug.  Will be uploading patch shortly.

Mike

P.S.  Fixed this while trying to reproduce the GParted crash.  Currently
watching a 50 GiB partition copy, so far successfully.
Comment 1 Mike Fleetwood 2013-04-13 12:15:40 UTC 
Created attachment 241444 [details] [review]
Full ntfs fix (v1)

Here's the fix.

P.S.  Copy of 50 GiB partition on my Fedora 14 desktop with GParted
master completed successfully.  Not reproduced the crash yet.
Comment 2 Mike Fleetwood 2013-04-14 14:11:48 UTC 
Created attachment 241505 [details] [review]
Full ntfs fix (v2)

The only difference from v1 to v2 is removal of unnecessary curly braces.
Comment 3 Curtis Gedak 2013-04-17 17:20:40 UTC 
Thank you Mike for finding this problem and providing a patch to fix it.

My tests of this problem have been successful and the code change looks good to me.

The patch in comment #2 has been committed to the gnome repository for inclusion in the next release of GParted.

The relevant git command can be viewed at the following link:

Correctly report usage of absolutely full NTFS (#697946)
https://git.gnome.org/browse/gparted/commit/?id=50ca2e5f135da759748e7298cc1ad40160441597
Comment 4 Curtis Gedak 2013-04-24 16:29:27 UTC 
The code change to address this report has been included in GParted 0.16.0 released on April 24, 2013.