Eliminate dead NULL-assignments on argvFree() and argiFree()

Clean up spec %files -f <manifest> processing
- Split the manifest reading into a helper function
- Fix filename memleak when the file couldn't be opened and
  also check for ferror() on return from the fgets() loop - fgets()
  returns NULL on both EOF and error, previously an error would've
  gone silently unnoticed.o

Remove pointless variable and a dead assignment to it

Clean up checkFiles() a bit
- Initialize variables on declaration, eliminate dead rc assignment

Eliminate bunch of obvious dead NULL-assingments from misc build parts

Rip hysterical hoop-jumping on spec/pkg cpiolist freeing
- Just call rpmfiFree() directly, wtf?

Eliminate dead NULL-assignments on parsePolicies() exit

Eliminate bunch of obvious dead NULL-assignments from build script code

Eliminate bunch of obvious dead NULL-assignments from rpm generation

Eliminate dead NULL-assignments in parseFiles()

Eliminate dead NULL-assignments from rpmSpecFree()

Eliminate dead NULL-assignments on OFI frees on spec parsing

Move build restriction header copying to local scope
- Eliminates accidental use of the variables, eliminating any
  need for NULL'ing the iterator

Clean up + plug memleak in parseDescription()
- Eliminate dead NULL-assignments of local variables on exit
- Move stringbuf free to exit, previously error on readLine() leaked
  memory here

Eliminate dead NULL-assignments on exit from parseScript()

Clean up + fix memleaks in readIcon()
- Assume failure and use single point of exit where all allocations
  are freed without dumb dead-assignments. Also fixes a
  leak from icon allocation when Fread() fails.

Eliminate bunch of obvious dead NULL assignments in preamble parsing

More dead NULL assignments of local variables in filelist processing

Eliminate dead NULL assignments on processMetadataFile() exit

Minor cleanup to doUntar() to eliminate dead assignments on free()

Clean up rpmfcGenerateDependsHelper() (aka external depgen) a bit
- Remove redundant NULL check, rpmfiNext() handles it
- Move variables to local scope(s) from function scope where
  appropriate, remove dead NULL-assignments on free
- Log the "Finding ..." messages /before/ executing the (possibly
  long-running) dependency generation helpers
- rpmExpand() never returns NULL, don't bother testing for it

Eliminate a few more dead rpmdsFree() NULL assignments
- No real chance of accidental misuse of freed data here...

Don't bother NULL'ing everything on rpmfcFree()
- Replace umphteen dead NULL-assignments with a trash-n-burn memset()
- Since we're already testing for NULL, dont free NULL fc (not
  that it matters)

Minor cleanup to rpmfcExec()
- remove redundant xav re-initialization
- dont bother NULL'ing local variables on exit

Minor cleanups to rpmfcGenerateDepends()
- init genConfigDeps on declaration
- remove redundant NULL-checks: rpmfiNext() and rpmdsNext() check
  for and handle NULL cleanly, no need to test here.
- remove pointless local fileattr variable

Eliminate some dead assignments on rpmdsFree()
- Move ds declaration to local scope so there's no chance of
  accidental use of already freed data, ds is not needed at
  function scope.

Add a helper function for creation namespaced dependency sets, use it

Minor cleanups to rpmfcHelper()
- If runCmd() returns NULL there's no point in further processing,
  rearrange things to avoid extra work in that case. As a side-effect
  this also silences a false-positive NULL-deference warning from clang.

Eliminate dead assignment, early exit means failure already

Cosmetics: rewrap/indent to fit 80 columns

Sanitize handlePreambleTag() error returns
- Single point of exit, assume failure. There are plenty of calls
  here which can fail, each with different int/rpmRC return code
  styles when we simply want an RPMRC_OK/RPMRC_FAIL for the return.
  Also eliminates some dead rc assignments.

Eliminate a couple more dead increments in spec parsing code

Eliminate bunch of dead increments in spec filelist parsing

Clean up rpmrc include processing a bit
- Eliminate pointless switch-case when we only care about one
  possibility here. Also eliminate a dead-increment which becomes
  more obvious now with the surroundings unobfuscated.

Eliminate two dead increments in rpmrc platform processing

Handle readlink() failure in genCpioListAndHeader() correctly
- It might be a rare condition but it /can/ happen, and previously
  that would cause '\0' written out of bounds (at negative array offset).
  Also leave room for the terminating '\0' at the end of buffer when
  calling readlink(), previously a link exactly the size of buffer
  would've been silently truncated.

Add test-cases for three RhBug:705115  scenarios

Add tests for multilib file state dependencies
- Testcase for the file state consideration implemented in commit
  566a15c9c08aa593d05e2f55f1c171a48bc1b1bc

Add two basic multilib conflict resolution tests
- Add colored binary packages (as there's no guarantee we can build
  them from test-suite), test that colored conflicts get resolved
  to preferred color.

Colored conflict resolution part II
- Same as commit 7bbc9b073d9204f6ffb8a1091c189b80029d27a9 but
  within the to-be-installed set

Both files must be colored for multilib conflict resolution (RhBug:705115)
- File conflicts should only be resolved to preferred color when
  both files are colored. Prior to this we would happily overwrite
  scripts with a clearly conflicting ELF binary.

Add -r flag to find-debuginfo.sh to invoke eu-strip --reloc-debug-sections.
- This is a new option to eu-strip that strips out/resolves relocations
  between .debug_* elf sections in ET_REL files. This can save a lot of
  space for kernel module.

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Disable all scriptlets and collections centrally on --test and --justdb
- These are already taken care of in other places but they're littered
  a little bit of here and there, eg in case of collections these
  modes are only skipped deep down in the plugin code. This should
  make the intended behavior more obvious upfront.

Permit %verifyscript from non-installed packages
- Verification from non-installed packages can be useful sometimes:
  http://lists.rpm.org/pipermail/rpm-maint/2011-May/003015.html
- %verifyscript is unlike all other scriptlets so might as well
  take some shortcuts and bypass rpmteProcess() & all, just
  create a lone rpmte, force the header we got and call rpmpsmRun()
  directly on it, bypassing rpmteProcess() which is only an
  unnecessary complication here (based on patch by Michael Schroeder)

Don't run collections on script stages like %pre/posttrans, ugh.

Issue an error on failure to replace original package on signing
- Previously any failure on replacing the original package with the
  newly signed one would silently fail, causing rather confusing
  behavior when eg attempting to (re)sign readable but not writable
  packages.

Add support for nested Lua macro expansion (RhBug:490740)
- Lift the printbuffer accounting out of rpmlua into a struct of
  its own (Funny thing, this looks a whole lot like the macro
  expansion buffer and Good Ole StringBuf Brothers ... Boys ... Mam.
  Unify them one of these days maybe)
- Replace the simplistic on/off printbuffer with a stack of buffers,
  fixup the lone caller to use the new internal push/pop API.

Plug a memory leak on Lua rpm.expand()

Bail out of debuginfo if stabs format encountered (RhBug:453506)
- The previous "silently ignore" policy produces bogus debuginfo
  packages on some architectures and fails with other mysterious
  errors on others, better just fail hard until (if ever) somebody adds
  stabs support.

Fix %prep parse error to abort build
- Previously in some cases parse error in %prep could emit an error msg
  but still continue building due to error code bogosity. Clean up
  the mess a bit: assume failure and actually return the res(ult) we
  calculated instead of nextPart.

Cosmetics: remove redundant/empty doxygen markup, comments etc
- No functional changes.

Remove unused length return variables on doHeaderUnload()
- doHeaderUnload() is safe to call with NULL lengthPtr, the length
  is only of interest to headerRead() and headerWrite()

Handle HEADERFLAG_SORTED bit correctly in headerUnsort()
- Within rpm there's exactly one caller of headerUnsort() which has
  kinda taken care of re-sorting the header on exit, but only if it
  returns successfully, meaning the header sort status could've been
  left in inconsistent state with implications on consequent
  operations on that header. Also this is part of the public API
  yet callers have no chance of adjusting the flag when they call it
  (and why should they).
- Also dont bother sorting if the header is already in unsorted state.

Return explicit NULL on errors + frees in header code

Do not die on empty changelog section

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Distinguish between macro primitives with and w/o arguments
- do argument presence checks only for those requiring arguments

Add support for fileattr dependency namespacing
- This allows automatically wrapping generated dependencies into
  macro-expanded namespaces, eg foo(depname) by specifying
  __attr_namespace in the file attribute definition.
- Current generator scripts hardcode their namespaces but doing this
  on rpm-level gives extra flexibility eg when building for alternative
  versions/environments and frees generators from having to take care of
  the formatting.

Whoops, flags needs to be sorted for argvSearch() to work correctly

Unify fileattr include- and exclude-rule handling
- Handling both rule-types identically not only makes things more
  consistent but also adds exclude_flags support practically for free:
  "exeonly" and "magic_and_path" are usable for excludes too now, ditto
  for any other flags that might be added in the future.

Don't fetch missing sources on spec query

Unbreak %if, %if(n)arch, %if(n)os handling
- having those expressions without argument is no more allowed
- white space between expression and argument is mandatory
- rpmbuild no more confuses %ifabcd macro with %if abcd
- fix criptic message in case of invalid %if condition

-D is for --define, not --predefine (RhBug:706161)

Support "magic_and_path" flag in fileattrs
- Allow "magic_and_path" flag to configure that files must
  match both regexpes to be sent to the dependency generator.

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Remove redundant NULL checks
- Fclose() handles NULL by itself, no need to check in callers.
  Also no need to NULL'ify the local variables here.

Fix error handling in payload copying
- Lift the payload copy into a separate helper function to clean
  things up a bit
- The code to handle read errors was unreachable before this,
  causing any read error (rare as they might be) to be silently
  ignored and package generation considered successful.
- This could/should perhaps use ufdCopy() instead but that ha
  its own set of problems (not clean wrt large files, error reporting...)

Eliminate dead cpio_copy() + related stuff
- These code-paths cannot have been hit since readRPM() was
  killed, mop out the leftovers

Update translations to pick up changes from the script file cleanups

Further simplify & cleanup script file (error) handling
- Simply always call addFileToFoo() and handle no scriptfile condition
  as early success case there (and allocate sb only if actually needed).
- Consolidate error logging into addFileToTagAux(), removing the need
  for umphteen nearly identical rpmlog error messages + related logic.
- Eliminate the now pointless addFileToFoo() wrapper functions
  and just call the main thing directly.

Clean up addFileToTagFoo() helpers
- Centralize the work into addFileToTagAux(), turning the other
  two helpers into dumb wrappers with the only difference being
  append or no append. This simplifies and clarifies the exit
  codes, stringbuf allocations etc a good deal.

Minor cleanups to cpio_doio()
- Remove redundant rpmtsEmpty(), this is done from rpmtsFree() anyway
- Dont bother NULL'ing local variables at exit

Simplify rpmio flags handling in writeRPM() / cpio_doio()
- The payload macros are already expanded in writeRPM(), no point
  re-expanding in cpio_doio() and handle fallback to default gzdio
  in writeRPM() already in case the payload is not set in config
  or is bogus. Also rpmExpand() cannot return NULL so there's no
  point in checking for that.

Plug a minor memleak on writeRPM() error paths

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Eliminate bogus ferror() checks
- fopen() returns NULL on errors, never an opened stream with error
  flag set. These are leftovers from past where rpmio fd was used
  instead of FILE and probably the checks were bogus even back then too.

Always copy macro source when expanding it
- A macro can undefine itself, and unless we grab a copy of it we'll
  end up accessing already freed memory. Fixes a regression from
  commit ebc4ceaaeb8bb59019f4635471b28eb5f3eaaaa6 which assumed
  a copy is not always needed.

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Empty transaction is not an error (RhBug:699929)
- Empty transaction just means we have nothing at all to do,
  but it's not an actual error.

Do not abort if chown/chmod fails but the file is already correct
- This small patch makes rpm not abort the installation if
  chown()/chmod() failed but the files already have the correct
  ownership/mode. It also allows a failed mtime update on directories.

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Fix segfault on build with empty %prep section

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Fix "method not permitted before handle's open method" on --verifydb
- In verify mode, the db is not really opened, thus it's an error
  to call dbiFlags() which calls db->get_open_flags().

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Clean up file type verification logic a bit
- Non-regular files mostly share the same unverifiable properties,
  no point in listing all the cases separately. Links are a notable
  exception in that they're different from everything else, handle
  that separately.
- Also clean up other formatting: wrap lines at a better point +
  avoid multiline-comments when single line suffices.

Remove obsolete timedRead() from API

RIP rpmsqFork() + rpmsqWait() and the related bits
- Also remove additional thread protection: we're not supporting
  threads anywhere else either. If/when thread-protection is added,
  this is ulikely to be the first place anyway...

Use good ol' fork() + waitpid() for running scriptlets
- We have no need for anything fancier, this makes 80% of the
  overly complex & subtle semi-threaded and in places just broken
  code in rpmsq.[ch] unused and unnecessary.

Remove unneeded regex grouping.

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Comment spelling fix.

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Add lrzip support.

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Add lzip support.

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>

Remove duplicate fpLookup() doxygen annotation

Remove redundant strlen()'s on sha1 digest verification
- string comparison notices different length anyway, avoid going
  through the same data twice

Avoid malloc() + redundant strlen() calls in dbiFindByLabel()

Simplify findTag()

Fix find-lang so that it finds *@*.qm QT i18n files (RhBug:699945)

Understand 'PK00' zip archives (RhBug:699529)
- it is historically used by PKZIP when storing archive to removable
  media (floppy), some java apps still use it
- thanks to Karel Klic

Do not compile unused functions when SELinux support is disabled

Handle errors from moving target file into place in rpmSign()
- Signing isn't successful unless we manage to replace the original
  file with the signed one, take the stat() etc returns into count.

headerPut() and headerDel() returns aren't interesting here

Remove redundant indentation block from rpmSign()
- No code changes, just formatting sanity/prettiness

Just ignore miFreeHeader() return on rpmdbNextIterator() too
- There's no meaningful way to return a write-error here,
  rpmdbNextIterator() caller wouldn't know whether NULL at termination
  is early due to error or not. So just ignore the return from
  miFreeHeader(), it can only fail on rewrite mode (ie markReplacedFiles())
  so impact is rather limited anyway.

Ignore dbiSync() return code everywhere
- Since we're not checking the return then dont bother storing it either.
  The sync is fairly unlikely to fail if other operations succeed
  as sync failures generally revolve around invalid file descriptors etc.

Oops, thinko/typo in commit 8215e17c248f91f501bd8a2968dfadf896a350ce

Remove pointless xx tmp return code from dbiAppendSet() result