Fix out of bounds compilation warning

When compiling with -D_FORTIFY_SOURCE=2, gcc raises the following
warning:

    In file included from /usr/include/string.h:494,
                     from /build/libsolv/src/libsolv-0.6.35/src/policy.c:16:
    In function ‘memcpy’,
        inlined from ‘urpm_reorder.isra.18’ at /build/libsolv/src/libsolv-0.6.35/src/policy.c:1239:9:
    /usr/include/bits/string_fortified.h:34:10: warning: ‘__builtin_memcpy’ forming offset 8 is out of the bounds [0, 7] [-Warray-bounds]
       return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Culprit code is:

     memcpy(kn, "kernel", 8);
     memcpy(kn + 6, flavor, release - flavor + 1);

The first memcpy copies 8 bytes whereas the string "kernel" as only 7
bytes. It does not have serious consequence since the second memcpy
overwrites the unwanted byte. Anyway, it is better to fix it.

Add changes, bump version to 0.6.35

Fix indent

Make the arch score unsigned int

Bring id2color check in poolarch.c in sync with poolarch.h

Also use unsigned int when bounds checking pool_arch2color()

This is done so that the code doesn't crash when you pass
a rel id.

Refactor arch handling

lastarch is no longer the last element of id2arch, but now the
size of the array. This means that lastarch == 0 now reliable means
that no architecture was set.

To be compatible to old versions we over-allocate id2arch so that
id2arch[lastarch] is always zero.

There's also a new pool_arch2score function that abstracts accessing
the id2arch array.

Allow zchunk files with zero chunks

zchunk: fix bounds check in getuint

Refactor checksum parsing

Also fix number of chunks handling, it includes the first dictionary
chunk.

Adapt to zchunk format change

Add generated manpage as well

Fix RPMDB_KEEP_GPG_PUBKEY handling

Add manpage for solv tool

Eek, fix typo

Adapt for changed pubring naming in gpg

Fix getinstalledrpmdbids handling of the match attribute

We currently have to iterate through all package names
because rpm lacks an interface to just get one specific name.

no bindings for sles11

No python3 in SLE11

libzstd is only available for >= leap15

Add back missing changes line

Update the spec to current standards

Based in pull request 250 by Neil Gompa and Tomas Chvatal.

zchunk: support extracting of streams other than "1"

Fix parsing of susetags when the content file does not have a DESCRDIR entry

Build with zstd and zchunk support

Fix reading of content file in susetags

Drop the patchcheck tool

Nobody uses and maintains it anymore.

Add compat symlink for repo2solv

Move old repo2solv.sh into examples

Fix bug in susedata languages extension

Adding new repodatas may corrupt the current dataiterator,
so we need to first collect all entries and then add them
afterwards.

Add repo2solv manpage

Add new repo2solv tool replacing the old shell script

Add missing solv_zchunk.h file

Oops. Should look more carful at the untracked files list.

solv_zchunk: bite should be unsigned int

Get rid of some warnings

Support zchunk compression

Fix handling of deleted keys in repo_search when a keyname is given

Export repodata_set_kv and make use of it in repo_rpmdb.c

Support saving the solvable dependencies as normal idarray

The default is still relidarray, i.e. sorted arrays.

Support solvable dependency arrays of type REPOKEY_TYPE_IDARRAY

Refactor pool_resize_rels_hash

Refactor internal stringpool_resize_hash function

Support zstd compression

Merge pull request #274 from ignatenkobrain/solvable2id

add pool_solvable2id()

bindings: fix dir2str function return type

Fixes: 316c436de3b57e1c22d8368b366ef62a66d72e8e
Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

add pool_solvable2id()

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

Fix fp double close, work around some false positives from covscan

Merge pull request #273 from dbnicholson/deb-control-size

Bump debian control.tar size limit

Bump debian control.tar size limit

Increase the arbitrary control.tar limit from 1 MB to 16 MB. We came
across an openjdk-7-jdk package that had a 2.4MB control.tar.gz and
libsolv failed to parse it. dpkg doesn't actually apply a limit to these
files.

Tweak documentation of add_dirstr method

Merge pull request #271 from ignatenkobrain/python

bindings: expose repodata_str2dir, repodata_dir2str and repodata_add_…

bindings: expose repodata_str2dir, repodata_dir2str and repodata_add_dirstr

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

Remove wrong solv_free(data->vincore) in repodata_internalize

We're extending the old buffer, so the free is always wrong.

Merge pull request #262 from j-mracek/const

Add const for second variable

Merge pull request #266 from ignatenkobrain/typo

doc: fix typo

Add RPMDB_KEEP_GPG_PUBKEY flag

Add const for second variable

The second variable is used as a source for copy and it is not modified
therefore it should be marked as s const. Then it is more easy to identify what
is source and what is target and also it is clear that the source variable will
be not modify by the function.

Additionally it allows to use const map or queue to create a copy or other
operations.

doc: fix typo

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

Tweak source heuristic in ENABLE_RPMPKG_LIBRPM case

Use HEADERGET_MINMEM to make the code faster and save some memory.

Refactor repodata_set_kv function

Get rid of old cmake cruft

The -DVERSION definition is never used, so we can delete it plus
the cmake_policy setting.

Merge pull request #264 from m-blaha/rpmsourceheuristic

source/binary rpm detection heuristic when ENABLE_RPMPKG_LIBRPM used

source/binary rpm detection heuristic when ENABLE_RPMPKG_LIBRPM used

The headerIsSource() simply checks a presence of the RPMTAG_SOURCERPM
tag in package header, which is not really useful in this context.
This patch provides the same "heuristic" as the librpm uses for
detecting source/binary rpm.

Relevant bug: https://bugzilla.redhat.com/show_bug.cgi?id=1572338

Add changes, bump version to 0.6.34

Make sure the product file comes from /etc/products.d for the fallback product search

Also use suggests for ordering packages

Add changes, bump version to 0.6.33

Fix Selection.clone implementation and documentation

Use %license instead of %doc for the license

Merge pull request #257 from ignatenkobrain/seleciton-clone

bindings: use $self for Selection.clone()

Fix saving of branches when refining problems

This bug caused a mem-leak which led to its discovery...

bindings: use $self for Selection.clone()

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

Fix parserpmrichdep conditional

Merge pull request #256 from ignatenkobrain/python-rich

bindings: expose parserpmrichdep()

bindings: expose parserpmrichdep()

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

Work around rpm bug if HEADERIMPORT_FAST is used

With HEADERIMPORT_FAST, rpm uses the difference between offsets
to determine the entry length. Unfortunately, the length is
used as entry count for RPM_BIN_TYPE, so the cound may end up
to big. So trim it again to match the md5 length.

Thanks to Oleg Girko <ol@infoserver.lv> for investigating this.

Fix changes

Add changes, bump version to 0.6.32

Rework fetch phase in fileconflict handling

We now handle all files of one package in one go instead of
iterating over every file. In most cases this does not make
a difference, as the pruned candidate list is very small. But
it can be big if two packages with the same content get
installed, e.g. two kernels with same version but different
rebuild count.

Improve file conflict output in example solver

fileconflicts: make sure at least one candidate is a new install

No need to get only installed packages.

Remove solver_disableproblem and solver_enableproblem

They are internal to the solver and can't be used from
outside. Seems they were added to the libsolv.ver map by
accident.

Add changes, bump version to 0.6.31

Move new functions into SOLV_1.2 section

Use exact size types in header access functions

This matters for ENABLE_RPMPKG_LIBRPM, as we must match what
librpm returns.

Fix comment

Make serialize_dbenv_ops static

Some more code cleanup

Allow pool_error to be called with no pool

In that case we just return with the specified value and
don't log anything.

Move database functions in header files

I'm not completely happy with the result, but it's already
much nicer than before. And we can also habe repo_rpmdb_lmdb.h
if we must...

Also use headread if RPMPKG_LIBRPM is not defined

Fix memory leaks in LIBRPM code

Fix _dbpath macro definition

The rootdir must come after _dbpath...

Add ENABLE_RPMDB_LIBRPM and ENABLE_RPMPKG_LIBRPM options

Those two control if librpm should be used to read headers
from the rpm database and to get information from the headers.
They are currently a bit experimental, so use at your own risk.

This is based on patches from Florian Festi and Panu Matilainen.
Thanks!

Add pool_set_whatprovides function

Set an entry in the whatprovides table. Also make the pool methods
set_namespaceproviders and flush_namespaceproviders available in
the bindings.

Add map_clr_at and MAPCLR_AT

Clear some bits around the specified bit.

testcase_add_testtags: make sure that we have a solvable when adding data

Fix last commit when the involvedmap has a zero size

Set dup updaters when creating the dup maps

We used to do this when creating the dup rules. But that
means the update information is not available when creating
the update rules which results in no special update data
being saved. Fixes bnc#1074207.

Merge pull request #248 from termim/zlib-typo

fix typo