diff options
| author | unknown <guilhem@mysql.com> | 2005-01-06 21:30:39 +0100 |
|---|---|---|
| committer | unknown <guilhem@mysql.com> | 2005-01-06 21:30:39 +0100 |
| commit | 5b47aaed3a34c0b4c9f4c756dc7c65c8bb3a0393 (patch) | |
| tree | b3fd93e9768755a98f6b4005b0968a43dac6bd9c /mysys/my_write.c | |
| parent | 10844e41da65bd86a781e314a15a60b8b21d40d8 (diff) | |
| download | mariadb-git-5b47aaed3a34c0b4c9f4c756dc7c65c8bb3a0393.tar.gz | |
Fix for BUG#7714 "if disk full, sometimes MyISAM doesn't wait for free space, corrupts table"
This happened only if my_write() couldn't write even one byte.
I cannot easily add a .test for this, but I tested by hand before and after the change.
mysys/my_write.c:
Monty and I could not find a reason why a write should not wait for free disk space
(if disk is full) because it could not write at least one byte; doing so certainly
corrupts tables. my_pwrite() and my_fwrite() don't test for -1, so no problem.
Diffstat (limited to 'mysys/my_write.c')
| -rw-r--r-- | mysys/my_write.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/mysys/my_write.c b/mysys/my_write.c index da378d115f1..1d1a893090a 100644 --- a/mysys/my_write.c +++ b/mysys/my_write.c @@ -49,8 +49,7 @@ uint my_write(int Filedes, const byte *Buffer, uint Count, myf MyFlags) MyFlags&= ~ MY_WAIT_IF_FULL; /* End if aborted by user */ #endif if ((my_errno == ENOSPC || my_errno == EDQUOT) && - (MyFlags & MY_WAIT_IF_FULL) && - (uint) writenbytes != (uint) -1) + (MyFlags & MY_WAIT_IF_FULL)) { if (!(errors++ % MY_WAIT_GIVE_USER_A_MESSAGE)) my_error(EE_DISK_FULL,MYF(ME_BELL | ME_NOREFRESH), |