Overview

This is a patch release that fixes some minor bugs, tidies the code for many compiler warnings, and improves windows compatibility. Upgrading from v2.3.1 is recommended for most people, and essential for people using platforms experiencing bugs #214 or #207.

Included Changes

• Fix #214 heap corruption for too small kbloom. This could have crashed delta operations for very small files/signatures. Strangely it didn't seem to cause problems for most compilers/platforms, but did trigger errors for new versions of MSVC. (ljusten, https://github.com/librsync/librsync/pull/213)

• Fix #207 and add Travis Windows checks and improve compatibility. Turn on -Wconversion -Wno-sign-conversion warnings for clang. Add MSVC compiler flags to turn off posix warnings. Make all code compile clean with no warnings on all Travis platforms. Added cmake config checking for windows io.h and improve fileutil.c for MSVC. Fix broken error handling in rs_file_copy_cb(). Improved trace output, making it less spamy and more consistent. Add patch checking for invalid literal lengths. Improve internal variable and argument types. Add explicit type conversions. (dbaarda, https://github.com/librsync/librsync/pull/208)

• Fix a bug so patch will now fail returning RS_CORRUPT on encountering a zero length copy command instead of hanging. Make copy_cb() copying more data than requested an assert-fail on debug builds, and a log-warning for release builds. Make trace output a little less spammy about copy_cb() return values. (dbaarda, https://github.com/librsync/librsync/pull/206)

Known Issues

• The debian package configs are still very out of date.

• There is still no documentation on how to compile on or cross-compile for windows. See https://github.com/librsync/librsync/issues/171 for help.