Merge branch 'wip-signed-integer-overflow-cppcheck' into 'master'

Wip signed integer overflow cppcheck

Fix for cppcheck signed integer overflow error

See merge request !7

Resolve cppcheck Signed integer overflow error

The type of expression '1<<31' is signed int and this causes cppcheck to
issue the following warning.

src/gf_w32.c:681]: (error) Signed integer overflow for expression
'1<<31'.

Signed-off-by: Brad Hubbard <bhubbard@redhat.com>

Merge branch 'wip-documentation' into 'master'

jerasure.org is http, not https

See merge request !3

jerasure.org is http, not https

Also add a reference to the jerasure project @ jerasure.org

Signed-off-by: Loic Dachary <loic@dachary.org>

Merge branch 'wip-documentation' into 'master'

documentation

updates to replace URLs that do not contain useful information

See merge request !2

documentation: update with jerasure.org new home

Signed-off-by: Loic Dachary <loic@dachary.org>

Revert "Removed PDF from the repo and added a note in the README that describes how to"

This reverts commit f48b2622214084689d5a77dd290d374852e3cb29.

Merge pull request #2 from dachary/wip-exit

Merging a set of commits from Loic: Use assert(0) to exit(1) and clean-up some of the test code.

tests: add minimal encoder/decoder test

Add a test to run with make check to run encoder and decoder to make
sure they work at least in one simple case. It is also useful as a
documentation about how to use them.

Signed-off-by: Loic Dachary <loic@dachary.org>

tests: fail if gf_methods is not found

If the gf_methods was not found, the test would silently succeed doing
nothing. Check for existence and fail if it is not in the path.

Signed-off-by: Loic Dachary <loic@dachary.org>

decoder/encoder: fix compilation warnings

Resolve compilation warnings about unused variables and function return
values being ignored.

Signed-off-by: Loic Dachary <loic@dachary.org>

decoder: allow for path len > 100 characters

It's not that uncommon to have path longer than 100 characters.

Signed-off-by: Loic Dachary <loic@dachary.org>

use assert(0) instead of exit(1)

When a fatal error (unaligned memory etc.) is detected, jerasure
should assert(3) instead of exit(3) to give a chance to the calling
program to catch the exception and display a stack trace. Although it is
possible for gdb to display the stack trace and break on exit, libraries
are not usually expected to terminate the calling program in this way.

Signed-off-by: Loic Dachary <loic@dachary.org>

remove unused variable in tests

Signed-off-by: Loic Dachary <loic-201408@dachary.org>

define galois_uninit_field

To free resources allocated by galois_init_default_field.

Signed-off-by: Loic Dachary <loic-201408@dachary.org>

Adding option to disable SSE support in configure

Removed PDF from the repo and added a note in the README that describes how to
get the manual.

Merged in dachary/jerasure/wip-galois-init (pull request #22)

add galois_init_default_field error code

add galois_init_default_field error code

galois_init_default_field returns an errno(3) code in case of error
instead of exiting. This is handy when the caller needs to perform
cleanup or error reporting when an error occurs instead of exit(2).

The exit(2) based error handling is preserved in the static
galois_init() function which is used in galois.c instead and is based on
galois_init_default_field to avoid code duplication.

Signed-off-by: Loic Dachary <loic@dachary.org>

Merged in dalgaaf/jerasure/wip-da-coverity-rebase (pull request #21)

[UPDATED][Rebased #8] Fixes for SCA issues

jerasure.c: add more checks for talloc/malloc results

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>

jerasure.c: add check for result of malloc()

Add check for bitmatrix and return NULL if malloc failed.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>

jerasure.c: free memory before return in error case

Fix for Coverity issue from Ceph project:

CID 1093211 (#1 of 1): Resource leak (RESOURCE_LEAK)
 20. leaked_storage: Variable "ind_to_row" going out of scope leaks
 the storage it points to.

CID 1093212 (#1 of 1): Resource leak (RESOURCE_LEAK)
 20. leaked_storage: Variable "row_ids" going out of scope leaks the
 storage it points to.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>

Examples/*coder.c: add missing include of unistd.h

Fix warning: implicit declaration of function 'getcwd' is invalid in
C99 [-Wimplicit-function-declaration]

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>

Examples/reed_sol_time_gf.c: include sys/time.h

Fix warning: implicit declaration of function 'gettimeofday' is
invalid in C99 [-Wimplicit-function-declaration]

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>

jerasure.c: fix memory leak in error case

Check matrix for NULL before call talloc().

CID 1093213 (#1 of 1): Resource leak (RESOURCE_LEAK)
 4. leaked_storage: Variable "bitmatrix" going out of scope
 leaks the storage it points to.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>

Merged in dachary/jerasure/wip-init (pull request #17)

make galois_init_default_field(int w) extern

make galois_init_default_field(int w) extern

So that the application can initialize gfp_array and gfp_is_composite
instead of relying on initialization happening implicitly when the
multiply or xor functions are called. The init function can be called
once when the application guarantees thread safety. And the multiply and
xor functions can be called from multiple threads without risking races.

Signed-off-by: Loic Dachary <loic@dachary.org>

Merged in dachary/jerasure/wip-make-check (pull request #16)

run tests with make check

Merged in dwglessner/jerasure/pull-#9-cleanup (pull request #15)

AM_CFLAGS and typo cleanup

run tests with make check

* Update the README accordingly
* Add the VALGRIND variable to run thru valgrind where possible
* Add the make check files administrative files to .gitignore

Signed-off-by: Loic Dachary <loic@dachary.org>

Fix typo about Autoconf >= 2.65 requirement.

Remove -O3, -fPIC, $(INCLUDES) from AM_CFLAGS.

They aren't needed, and can be harmful if they override user selection.

Merged in dachary/jerasure/wip-gf-complete-tests (pull request #12)

allow override of GF-Complete directory in test_all_gfs.sh

Merged in dachary/jerasure/wip-warning-unused (pull request #14)

silence warning about bestrow used uninitialized

silence warning about bestrow used uninitialized

Signed-off-by: Loic Dachary <loic@dachary.org>

Merged in dachary/jerasure/wip-gitignore (pull request #11)

add .dirstamp to gitignore

allow override of GF-Complete directory in test_all_gfs.sh

And document how to run it in the README

Signed-off-by: Loic Dachary <loic@dachary.org>

add .dirstamp to gitignore

Signed-off-by: Loic Dachary <loic@dachary.org>

This is the squashed pull request from David Glessner, squashed
and reviewed by Kevin Greenan.

Updated README

Allow out-of-source builds. Quiet autogen.sh warnings.

Use AM_CPPFLAGS instead of INCLUDES. Use $(top_srcdir).

Add .gitignore to quiet git status.

Add project URL to AC_INIT.

Stop libtool from compiling files twice.

Have git ignore .deps/.

Don't override user CPPFLAGS.

(PIC options appear to be set already. INCLUDES already included.)

Clean configure.ac.

Make it closer to autoscan output.

Have autotools create INSTALL.

Use AC_MSG_FAILURE if GF-Complete not found.

Run autogen.sh.

(autoconf 2.69, automake 1.14.1, libtool 2.4.2)

Add some .gitignore files.

Fix configure cpuid unknown issue.

Move AX_EXT before AC_CHECK_LIB(gf_complete...) so that
-lgf_complete doesn't cause CPUID conftest compile to fail.

Don't check for internal gf_int.h header.

GF-Complete doesn't install it.

Quiet some autoreconf warnings.

INCLUDES is now AM_CPPFLAGS. Use single-argument AM_INIT_AUTOMAKE.

Remove some AC_REQUIRE whose conftest always failed.

Quiet configure warning.

(ARCH_64 doesn't appear to be used.)

CPUID "unknown" fix from upstream autoconf-archive.

Allow out-of-source builds. Use dependency tracking.

Remove config.h.in~.

Update ax_check_compile_flag.m4 from autoconf-archive.

Add .gitattributes.

Remove autoreconf-generated files.

Remove ACLOCAL_AMFLAGS.

See Automake 1.13 release notes.

Add files for 'make dist'.

Quiet some warnings.

getcwd() in <unistd.h>. Remove some vars.

Make headers compatible with C++.

Quiet some warnings.

Install additional headers in include/jerasure/.

Quiet some configure check internal failures.

Use new AX_REQUIRE_DEFINED instead of AC_REQUIRE, which expands
the macro and causes internal compile failure.

Fix file permissions.

Remove INTEL_SSE compiler defines.

(Can use HAVE_xxx or __xxx__ instead.)

Set default CFLAGS to '-g -O3 -Wall'.

Add more checks from autoscan.

Use AC_CONFIG_AUX_DIR([build-aux]).

Use processor time for timing.

Use clock() instead of gettimeofday().

Use common LDADD in Makefile.am.

Remove pre-autotools makefiles.

Ignore *.a. (Missed earlier.)

Quiet more warnings.

Document need for autoreconf -fi.

Removed README.nd and README.txt so changes don't need to be duplicated.

Remove autogen.sh.

Just use "autoreconf --force --install".

Quiet configure warning.

(ARCH_64 doesn't appear to be used.)

Merged in dachary/jerasure/wip-compilation-warnings (pull request #2)

fix compilation warnings

add missing return value to functions that require it

Signed-off-by: Loic Dachary <loic@dachary.org>

Fixed some compiler warnings.

Added compilation instructions for when you don't have GF-Complete
installed as root.

Added the user manual PDF to Manual.pdf