Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Added io checks when writing checksum into rdb. #857

Closed
wants to merge 1 commit into from

1 participant

@yoav-steinberg

Check for io errors when writing the checksum to an rdb.

@mattsta mattsta referenced this pull request from a commit in mattsta/redis
@yoav-steinberg yoav-steinberg Add error check for writing RDB checksum
Closes #857
80d1f8f
@mattsta mattsta referenced this pull request from a commit in mattsta/redis
@yoav-steinberg yoav-steinberg Add error check for writing RDB checksum
Closes #857
4c75bf0
@mattsta mattsta referenced this pull request
Closed

ALL simple issue fixes #1906

@mattsta mattsta referenced this pull request from a commit in mattsta/redis
@yoav-steinberg yoav-steinberg Add error check for writing RDB checksum
Closes #857
87d3c48
@antirez antirez closed this in 0a98b21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 1, 2013
  1. @yoav-steinberg

    Added io checks when writing checksum into rdb.

    yoav-steinberg authored
    Conflicts:
    
    	src/rdb.c
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  src/rdb.c
View
2  src/rdb.c
@@ -687,7 +687,7 @@ int rdbSave(char *filename) {
* loading code skips the check in this case. */
cksum = rdb.cksum;
memrev64ifbe(&cksum);
- rioWrite(&rdb,&cksum,8);
+ if (rioWrite(&rdb,&cksum,8) == 0) goto werr;
/* Make sure data will not remain on the OS's output buffers */
fflush(fp);
Something went wrong with that request. Please try again.