diff options
| -rw-r--r-- | meta/recipes-devtools/rsync/rsync-3.1.1/rsync.git-eac858085.patch | 101 | ||||
| -rw-r--r-- | meta/recipes-devtools/rsync/rsync_3.1.1.bb | 1 |
2 files changed, 102 insertions, 0 deletions
diff --git a/meta/recipes-devtools/rsync/rsync-3.1.1/rsync.git-eac858085.patch b/meta/recipes-devtools/rsync/rsync-3.1.1/rsync.git-eac858085.patch new file mode 100644 index 0000000000..1fcac490ae --- /dev/null +++ b/meta/recipes-devtools/rsync/rsync-3.1.1/rsync.git-eac858085.patch | |||
| @@ -0,0 +1,101 @@ | |||
| 1 | From eac858085e3ac94ec0ab5061d11f52652c90a869 Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Wayne Davison <wayned@samba.org> | ||
| 3 | Date: Mon, 11 May 2015 12:36:20 -0700 | ||
| 4 | Subject: [PATCH 1/1] Add compat flag to allow proper seed checksum order. | ||
| 5 | Fixes the equivalent of librsync's CVE-2014-8242 issue. | ||
| 6 | |||
| 7 | Upstream-Status: Backport | ||
| 8 | |||
| 9 | Signed-off-by: Roy Li <rongqing.li@windriver.com> | ||
| 10 | --- | ||
| 11 | checksum.c | 17 +++++++++++++---- | ||
| 12 | compat.c | 5 +++++ | ||
| 13 | options.c | 1 + | ||
| 14 | 3 files changed, 19 insertions(+), 4 deletions(-) | ||
| 15 | |||
| 16 | diff --git a/checksum.c b/checksum.c | ||
| 17 | index a1c2aa2..933b514 100644 | ||
| 18 | --- a/checksum.c | ||
| 19 | +++ b/checksum.c | ||
| 20 | @@ -23,6 +23,7 @@ | ||
| 21 | |||
| 22 | extern int checksum_seed; | ||
| 23 | extern int protocol_version; | ||
| 24 | +extern int proper_seed_order; | ||
| 25 | |||
| 26 | /* | ||
| 27 | a simple 32 bit checksum that can be upadted from either end | ||
| 28 | @@ -54,10 +55,18 @@ void get_checksum2(char *buf, int32 len, char *sum) | ||
| 29 | if (protocol_version >= 30) { | ||
| 30 | uchar seedbuf[4]; | ||
| 31 | md5_begin(&m); | ||
| 32 | - md5_update(&m, (uchar *)buf, len); | ||
| 33 | - if (checksum_seed) { | ||
| 34 | - SIVALu(seedbuf, 0, checksum_seed); | ||
| 35 | - md5_update(&m, seedbuf, 4); | ||
| 36 | + if (proper_seed_order) { | ||
| 37 | + if (checksum_seed) { | ||
| 38 | + SIVALu(seedbuf, 0, checksum_seed); | ||
| 39 | + md5_update(&m, seedbuf, 4); | ||
| 40 | + } | ||
| 41 | + md5_update(&m, (uchar *)buf, len); | ||
| 42 | + } else { | ||
| 43 | + md5_update(&m, (uchar *)buf, len); | ||
| 44 | + if (checksum_seed) { | ||
| 45 | + SIVALu(seedbuf, 0, checksum_seed); | ||
| 46 | + md5_update(&m, seedbuf, 4); | ||
| 47 | + } | ||
| 48 | } | ||
| 49 | md5_result(&m, (uchar *)sum); | ||
| 50 | } else { | ||
| 51 | diff --git a/compat.c b/compat.c | ||
| 52 | index 2454937..f89d466 100644 | ||
| 53 | --- a/compat.c | ||
| 54 | +++ b/compat.c | ||
| 55 | @@ -27,6 +27,7 @@ int inc_recurse = 0; | ||
| 56 | int compat_flags = 0; | ||
| 57 | int use_safe_inc_flist = 0; | ||
| 58 | int want_xattr_optim = 0; | ||
| 59 | +int proper_seed_order = 0; | ||
| 60 | |||
| 61 | extern int am_server; | ||
| 62 | extern int am_sender; | ||
| 63 | @@ -78,6 +79,7 @@ int filesfrom_convert = 0; | ||
| 64 | #define CF_SYMLINK_ICONV (1<<2) | ||
| 65 | #define CF_SAFE_FLIST (1<<3) | ||
| 66 | #define CF_AVOID_XATTR_OPTIM (1<<4) | ||
| 67 | +#define CF_CHKSUM_SEED_FIX (1<<5) | ||
| 68 | |||
| 69 | static const char *client_info; | ||
| 70 | |||
| 71 | @@ -271,12 +273,15 @@ void setup_protocol(int f_out,int f_in) | ||
| 72 | compat_flags |= CF_SAFE_FLIST; | ||
| 73 | if (local_server || strchr(client_info, 'x') != NULL) | ||
| 74 | compat_flags |= CF_AVOID_XATTR_OPTIM; | ||
| 75 | + if (local_server || strchr(client_info, 'C') != NULL) | ||
| 76 | + compat_flags |= CF_CHKSUM_SEED_FIX; | ||
| 77 | write_byte(f_out, compat_flags); | ||
| 78 | } else | ||
| 79 | compat_flags = read_byte(f_in); | ||
| 80 | /* The inc_recurse var MUST be set to 0 or 1. */ | ||
| 81 | inc_recurse = compat_flags & CF_INC_RECURSE ? 1 : 0; | ||
| 82 | want_xattr_optim = protocol_version >= 31 && !(compat_flags & CF_AVOID_XATTR_OPTIM); | ||
| 83 | + proper_seed_order = compat_flags & CF_CHKSUM_SEED_FIX ? 1 : 0; | ||
| 84 | if (am_sender) { | ||
| 85 | receiver_symlink_times = am_server | ||
| 86 | ? strchr(client_info, 'L') != NULL | ||
| 87 | diff --git a/options.c b/options.c | ||
| 88 | index 19c2b7d..4128b59 100644 | ||
| 89 | --- a/options.c | ||
| 90 | +++ b/options.c | ||
| 91 | @@ -2503,6 +2503,7 @@ void server_options(char **args, int *argc_p) | ||
| 92 | #endif | ||
| 93 | argstr[x++] = 'f'; /* flist I/O-error safety support */ | ||
| 94 | argstr[x++] = 'x'; /* xattr hardlink optimization not desired */ | ||
| 95 | + argstr[x++] = 'C'; /* support checksum seed order fix */ | ||
| 96 | } | ||
| 97 | |||
| 98 | if (x >= (int)sizeof argstr) { /* Not possible... */ | ||
| 99 | -- | ||
| 100 | 1.9.1 | ||
| 101 | |||
diff --git a/meta/recipes-devtools/rsync/rsync_3.1.1.bb b/meta/recipes-devtools/rsync/rsync_3.1.1.bb index 3a79154511..c74cdda943 100644 --- a/meta/recipes-devtools/rsync/rsync_3.1.1.bb +++ b/meta/recipes-devtools/rsync/rsync_3.1.1.bb | |||
| @@ -3,6 +3,7 @@ require rsync.inc | |||
| 3 | 3 | ||
| 4 | SRC_URI += "file://acinclude.m4 \ | 4 | SRC_URI += "file://acinclude.m4 \ |
| 5 | file://0001-Complain-if-an-inc-recursive-path-is-not-right-for-i.patch \ | 5 | file://0001-Complain-if-an-inc-recursive-path-is-not-right-for-i.patch \ |
| 6 | file://rsync.git-eac858085.patch \ | ||
| 6 | " | 7 | " |
| 7 | 8 | ||
| 8 | SRC_URI[md5sum] = "43bd6676f0b404326eee2d63be3cdcfe" | 9 | SRC_URI[md5sum] = "43bd6676f0b404326eee2d63be3cdcfe" |
