Merge pull request #290 from manish3kumar/integrate-sa

Integrate cppcheck with travis-ci for libexpat

Integrate cppcheck with travis-ci for libexpat

Signed-off-by: Manish, Kumar <manish3.kumar@intel.com>

Merge branch 'travis-clang-9'

Travis CI: Use Clang 9 on Linux

qa.sh: Dump compiler version

AppVeyor|Travis: Re-introduce clone depth

Depth 1 made surpassed pushed fail so it was dropped.
No depth limit wastes time and hardware resources though, and
has some risk that depth 1 will be re-introduced, by mistake.
So let's try 50 now.

Related commits:
- 10c235458830fc894a6381cf6267246c68848adf — AppVeyor
- 88cf657da91ef7a83d705cf7951cfd6c6b7d98fc — Travis

Merge branch 'solve-warnings.marcomaggi-2018-10-30' (fixes #235)

fixed warning about potential null pointer dereference

runtests.c: Address warnings about variable shadowing

Merge branch 'marcomaggi-cleaner-test-for-mingw.marcomaggi-2018-10-28' (fixes #231)

configure.ac: Use curly braces for variable

configure.ac: Adhust spelling of MinGW

cleaner test for mingw as host system

Travis CI: Drop "git.depth: 1" because subsequent pushes were failing ongoing builds

See 10c235458830fc894a6381cf6267246c68848adf for AppVeyor and more details.

fixed use of compilation flags in coverage script (#229)

Merge branch 'marcomaggi-shell-scripts-review.marcomaggi-2018-10-21' (fixes #229)

clean_coverage.sh: Use portable shebang

added shebang and comments to clean_coverage script

internal.h: Fix UNUSED_P for GCC 3.3 (fixes #288)

runtests.c: Address cppcheck warning (fixes #283)

[tests/runtests.c:647]: (error) Address of local auto-variable assigned to a function parameter.

AppVeyor: Drop "clone_depth: 1" because subsequent pushes were failing ongoing builds

Symptom:
  Build started
  git config --global core.autocrlf input
  git clone -q --depth=1 --branch=master https://github.com/libexpat/libexpat.git c:\projects\libexpat
  git checkout -qf d4d8d486af1810bff52612d84533985b9b47b7a6
  fatal: reference is not a tree: d4d8d486af1810bff52612d84533985b9b47b7a6

Cause:
  A more recent push made d4d8d486af1810bff52612d84533985b9b47b7a6
  the second-last commit so that --depth=1 would create a commit
  graph without that commit.

Documented at:
https://www.appveyor.com/docs/how-to/repository-shallow-clone/#setting-depth-of-git-clone-command

xmlwf: Ensure that STDMETHODIMP_ is defined for cppcheck (fixes #285)

xmltok.c: Use memcpy instead of copying single bytes (fixes #282)

Fixes cppcheck's complaint as a side-effect.

tests: Silence cppcheck false positive (fixes #284)

Merge branch 'MohammedKhajapasha-issue_273' (#281, fixes #273)

Travis: Limit distcheck tests to plain distcheck

--enable-xml-attr-info is tested elsewhere and this
combination does not seem to provide additional value
to justify runtime.

Travis: Drop test of in-source CMake

Out-of-source CMake is still there and is more important.

Enable CI for XML_ATTR_INFO

Enable CI to cover XML_ATTR_INFO feature.

Signed-off-by: Mohammed Khajapasha <mohammed.khajapasha@intel.com>

Merge branch 'rewrite-qa-sh' (#278)

Travis: Move from exclusion to inclusion approach

Travis: Excluding UndefinedBehaviorSanitizer for macOS due to link errors

https://travis-ci.org/libexpat/libexpat/jobs/556567990#L529

Travis: Disable gcov mode on macOS for lack of -s argument

gcov: Unknown command line argument '-s'

Travis: Fix MemorySanitizer exclude for macOS

qa.sh: Pass -fsanitize=undefined to linker as well

qa.sh: Use clang++ to link (for ubsan)

qa.sh: Support GCC coverage reporting with Apple GCC as well

Travis: Cover MemorySanitizer and UndefinedBehaviorSanitizer

qa.sh: Re-write

- A bit more flexible and high level
- Drop support for ncc
- Fix coverage rendering, drop split mode

Merge branch 'kkkunche-issue_239' (#239, #277)

Changes: Document #239 and #277

CMake: Write expat_config.h as late as needed

Previously, our post-processing of HAVE_GETRANDOM did
was not applied to expat_config.h.

CMake: Resolve code duplication by use of a macro

CMake: Support bypassing auto-detection of syscall SYS_getrandom

CMake: Turn getrandom build flag into a tribool

CMake: Move reset of HAVE_GETRANDOM after ConfigureChecks.cmake

CMake: Option to Disable the usuage of getrandom after autodetect

Merge branch 'issue-268-address-variable-shadowing' (#270, fixes #268)

Changes: Mention #268 and #270

Changes: Mention #244

Merge branch 'MohammedKhajapasha-master' (#264)

Changes: Document #264

coverage.sh: Simplify if statement

Conistent with unicode_enabled check above

coverage.sh: Make xml_attr_info_enabled match unicode_enabled

configure: Make --enable-xml-attr-info help match --enable-static

Enable XML_ATTR_INFO in build

enable XML_ATTR_INFO in build to cover XML_ATTR_INFO in CI

Signed-off-by: Mohammed Khajapasha <mohammed.khajapasha@intel.com>

xmlparse.c: Limit scope of new_connector to save sizeof(char*) on the stack

xmlparse.c: Drop needless re-calculation of keep (#268)

Fixes variable shadowing as a side effect.

xmlparse.c: Resolve temp variable shadow (#268)

Also add const.

Changes: Attach CVE-2018-20843 to #186 and #262

Changes: Document #265

Merge pull request #265 from xantares/patch-1

Fix link step on MinGW for CMake

Fix link step on MinGW

.def files are handled differently on MinGW than MSVC for the WIN32 target:

```
[ 14%] Linking C shared library libexpat.dll
/usr/lib/gcc/x86_64-w64-mingw32/9.1.0/../../../../x86_64-w64-mingw32/bin/ld: ../lib/libexpat.def:4: syntax error
/usr/lib/gcc/x86_64-w64-mingw32/9.1.0/../../../../x86_64-w64-mingw32/bin/ld:../lib/libexpat.def: file format not recognized; treating as linker script
/usr/lib/gcc/x86_64-w64-mingw32/9.1.0/../../../../x86_64-w64-mingw32/bin/ld:../lib/libexpat.def:1: syntax error
collect2: error: ld returned 1 exit status
```

Bump version from 2.2.6 to 2.2.7

Bump version info from 7:8:6 to 7:9:6

Changes: Group/re-order changes for upcoming 2.2.7

Changes: Document #158 and #263

Merge pull request #263 from kkkunche/master

Correct the man page installation path in CMake Build

Correct the man page installation path in CMake Build

CMake build system : corrected the man page creation to PROJECT_BINARY_DIR
and avoid creation into PROJECT_SOURCE_DIR.

Signed-off-by: kkkunche <kishore.kunche@intel.com>

Merge branch 'issue-186-fix-extraction-of-namespace-prefix'

Changes: Document #186 and #262

xmlparse.c: Fix extraction of namespace prefix from XML name (#186)

Merge branch 'issue-257-portable-shebang' (#257)

Changes: Document #257

Use "/usr/bin/env bash" shebang everywhere for protability (#257)

Fix build on OpenBSD (GH #257)

Travis CI: Install lzip on macOS for "make dist" (#258)

Brewfile: Sort

Travis CI: Install lzip for "make dist" (#258)

Changes: Remove @ GitHub username marker noise

Autotools: Produce .tar.{gz,lz,xz} release archives (#236, #258)

Merge pull request #250 from luzpaz/README-repology

README: Add Repology badge showing repos package is available in

README: Add Repology badge showing repos package is available in

Merge branch 'osx-travis-homebrew' (#152, #249)

qa.sh: Link with clang for Address Sanitizer

qa.sh: Dump config.log for configure failures

Travis CI: Keep macOS $PATH away from Linux

Travis CI: Test OSX

Use Brewfile for dependencies

qa.sh: Fix incomplete link flags for Address Sanitizer

Prefix "-Wc," is fot libtool, see
https://www.gnu.org/software/libtool/manual/libtool.html#Stripped-link-flags

Merge branch 'issue-247-fix-configure-without-docbook' (fixes #247)

Fix compilation for lack of docbook2x-man

The guard in doc/Makefile.am was supposed to protect against
publishing release tarballs lacking a compiled xmlwf.1 man page
without noticing.  Now it does no more than advertised.

Merge branch 'issue-245-visibility-check' (fixes #245)

Changes: Mention PR #246

m4: Fix check for -fvisibility=hidden for Clang

m4: Reformat C test code to be readable

Merge branch 'sdroege-win32-def-file-syntax'

Changes: Document fix to #243

Fix syntax of Windows module definition files

Otherwise this fails with:
  Cannot export @: symbol not defined

Merge pull request #237 from oshogbo/master

Fix compilation for -Werror with -Wunused-function/-Wall

 Un-break build libexpat.

The writeRandomBytes_arc4random is not used if the arc4random_buf
is available. This caused compiler to throw warnings.
Finally this breaks a build when warnings are treated as errors.

Merge branch 'KangLin-master' (#227)