Compare pointers against NULL, not 0

Have BDB use our allocation routines now that we can
- rfree() prototype doesn't match free() but it doesn't matter in this
  case as it's just a convencience wrapper with no extra functionality
  and setting NULL is ok according to BDB docs
- db->set_alloc() can't be used if environment is used, fix the condition

Add runtime settable custom memory allocation failure callback routine
- lets API users perform theirn own cleanup / exit through their own
  routines in case of allocation failure, or even try to free up
  some memory and retry allocation

Always use build + use our own allocator functions
- Consistent across platforms and will allow some things macros dont and
  memory checkers these days are smart enough to get decent output anyway
- Minimal namespacing with r-prefix, add compatibility macros to system.h
  for now so we dont have to change the entire codebase for this
- Also make rpmutil.h where the declarations and gcc __attribute__ macros
  are available everywhere

Eliminate unnecessary variable

Remove unused local variable

Remove db environment on close on chrooted operations
- As we open the db from outside the chroot, the environment ends up
  containing paths that are not valid once we enter the chroot causing
  dumb issues like RhBug:513699 and RhBug:507309.
- We'd be better off removing the environment always after access but
  limiting to chroots for now as the rpmdb open/close path is full of races
  when environment is not present. Chroots are somewhat special environemnts
  anyway and typically not concurrently accessed so it's less of an issue
  there.
- While this still has all sorts of issues, it at least leaves the rpmdb
  in chroot in a functional state after initial install.

Move the default transaction lock out of BDB environment namespace
- We dont want the transaction lock to get mixed with the BDB environment,
  nor do we want it wiped out with it as the ts lock protects things
  *around* the environment. Unlike the environment, any fcntl locks on
  the file get released when process dies, so if there is a lock,
  there's a process alive holding it and you'd better not remove it.
- Use the same lock path in macro configuration and built-in fallback
- Make it a hidden file to avoid people confusing lock file existence
  with it actually being locked

There's a function to test string equivalence...

Throw out useless db statistics structs
- besides being completely unused, these structs are defined by BDB
  public headers, no need to mirror them in rpm

Throw out pile of unused "stuff" from 2002

Minor cleanup to newRpmdb()
- no need to free what hasn't been allocated yet

One complicated way to write xstrdup() .. pff
- sqlite backend uses dbi subfile for table name, leave it alone although
  it is the same as the file name...
- allocate + free both file and table name separately

Eliminate long since unused db subfile from bdb code

Remove unnecessary arguments to db init and fini
- bdb environments are per db_home, not per file so dbfile etc dont make
  sense here

Also skip to be erased files in "netsharedpath"
fixes rhbz#494640

Initialize rpm earlier in rpm2cpio (RhBug:523260)
- url retrieval requires macros to be loaded

Take non-gi errors into account too (SuseBug:527191)
- patch from OpenSUSE / Michael Schroeder

Allow absolute paths in file lists again (SuseBug:535594, RhBug:521760)
- patch from OpenSUSE / Michael Schroeder
- build-time generated file lists should be placed in the build directory,
  but at least one valid use case for this is things like %files -f %{SOURCE10}

find-debuginfo.sh: use permissions 444 for all .debug files

Signed-off-by: Roland McGrath <roland@redhat.com>

Don't add dependency type into file requires and provides (RhBug:523282)
- unintended change in commit 3d8656f040d2c780c88d0cd63594898f0605a6fa

Clean up rpmalAllFileSatisfiesDepend() a bit
- avoid multiple strrchr() calls
- avoid _constfree(), the string size is known from strrchr() so we dont
  need to allocate on heap at all

Single point of exit for doSetupMacro() to clean up freeing allocations

Oops, accessing the wrong header in processBinaryFiles()
- dumb mistakes from commit 4177fdffd6e7414123062e8716afb22596608531

Rip the broken asynchronous psm machinery
- it was only "used" for the ldconfig optimization and actually been
  completely broken since the ordering rewrite

Rip the broken ldconfig optimization hack (RhBug:513224)
- it doesn't optimize much and what's worse, it sometimes optimizes out
  calls that would've been necessary

Rip unnecessary headerDump() and the last ancient artifacts using it
- xml format dump gives an output that can actually be further processed

Rip unused and useless headerGetLangs()
- nothing in the world uses it and the info is available through
  regular headerGet() anyway

Rip unused and useless headerUsageCount()

Eliminate unused goo from findEntry()

Handle NULL header in doHeaderUnload()
- avoids having to separately check elsewhere and avoids segfaulting
  on headerUnload() and headerReload() on NULL headers

Eliminate pointless increment

Eliminate yet more unused calculations from header code...

Eliminate unused pad calculations from header code

Eliminate yet another unused variable in header code

Eliminate last remaining leftovers from "8/98 bug" workaround
- the ieprev calculations aren't used for anything, should've been in
  commit 4d0d54492c7d539cc806ca9e7fa59e919c28e45d really

Eliminate unnecessary variable & related goo from regionSwab()

Initialize diff on declaration where it fits comfortably on one line

Move variable to block local scope, not needed elsewhere

Remove unnecessary assignment, diff is block local scope here

Reduce the number of exit points in parseFormat() to error and ok cases
- avoid a dozen redundant freeFormat() calls and dont bother NULLing
  local variable at the end

Add -Wempty-body to default options if gcc supports it
- ...to help protect against screwups like
  2ee45ef12aaa958b78c56b6d83bb18aa6a477d15

Eliminate unused variable

Eliminate unused variable

Minor rpmfcGenerateDependsHelper() cleanups
- initialize at declaration time, remove unnecessary variables etc

Move rl to the scope where its needed, dont bother assigning NULL

Single point of exit for doPatch() to clean up freeing

Eliminate bogus variable initialization
- i isn't used outside the loop at the end, move it there...

Add noreturn annotations to javadeps

Help compilers/analyzers a bit with luaL_error()
- luaL_error() never returns but as other tools dont know this it
  raises false alarms

Add noreturn annotations to argerror() helper functions

Add noreturn annotation to die() .. which itself should die

Avoid theoretical NULL pointer dereference
- if sbp ends up not set it's an "internal error" really, dont bother
  with error message

Avoid couple of (theoretical) NULL pointer dereferences

Remove bogus non-null check
- rpmdsInit() only returns NULL if passed NULL, which can't happen
  as ods is already verified to be non-NULL

Ugh, erroneously introduced ; busting the conditional
- broken long time ago in commit 2ee45ef12aaa958b78c56b6d83bb18aa6a477d15

Avoid potentially calling a NULL function pointer

Eliminate unused calculations

Eliminate unused variable

Eliminate bunch of unused variables

Eliminate unused variable and pointless null-check

Fix dumb some dumb leaks in python spec code

Eliminate unused counter variable

Remove unused initialization

Remove redundant reinitialization, cleanup

Add RPM_GNUC_NONNULL macro for portable use of gcc nonnull attribute
- gcc itself can only warn on literal NULL's passed where nonnull is
  expected but static analyzers can make better use of the info

Fix memleak in headerGetDumber() .. err .. Number()

Handle .desktop files with spaces in filename (rhbz#520920)
- thanks to Edward Sheldrake

Use %{nvra} as default query format, much simpler...

Fix dumb thinko from commit 4177fdffd6e7414123062e8716afb22596608531

Take advantage of headerGetNumber() to get buildtime..

Further simplify rpmVersionCompare()
- headerGetNumber() side-effect of returning zero on errors fits this
  job perfectly, simplifying things somewhat

Deprecate headerGetColor(), implement using headerGetNumber()

Eliminate headerGetColor() uses everywhere

Add header color tag extension, ie headerGetColor()
- use common numeric tag generation helper for header color
  and dbinstance tags

Add convenience function for retrieving numeric header data

Oops, dont leak memory on invalid types to headerGet[As]String()

Take advantage of new header string getters

Clean up rpmtsAddInstallElement() a bit
- only bother retrieving os, arch and colors if actually needed
- obsoletes are never colored, remove the related cruft

Take advantage of new header string getters

Take advantage of new header string getters

Take advantage of new header string getters

Simplify rpmpsmStage() and srpm install a bit with the new getters

Simplify rpmVersionCompare() a bit

Simplify headerCheckPayloadFormat() a bit

Deprecate headerNVR() and friends
- reimplement using the new generic getters to simplify
- mark as deprecated, no uses inside rpm remain

Eliminate uses of headerNVR() and friends in librpm (and python) code

Eliminate uses of headerNVR() and friends in librpmbuild code

Add tag extensions for common name-[epoch:]version-release[.arch] combos
- RPMTAG_NVRA same as rpm5.org, others are new tags

Add two new convenience functions for header string data
- headerGetString() for retrieving const strings directly from header memory
- headerGetAsString() for converting any non-array tag to string

Add dbinstance tag extension (port from rpm5.org)

Group tag inheritance doesn't need special treatment, just copy with others

Triggerprein has been implemented since January 2008, update comment

Add provide-extractors for fonts and desktop files
- extract mime handler capabilities from application .desktop files
- extract font family names and supported languages from font files
- both are used by PackageKit for on-demand installing of fonts and
  applications, but are useful for other things too

Further file-classifier tweaks
- disable "known tokens" lookups in ascii files, this is way too unreliable
  to do anything but cause pain
- consider text files as RPMFC_TEXT consistently
- font metrics files are not fonts

Make internal file classifier look into compressed files
- prerequisite for various things like font provide extraction
- at least libmagic from file-5.00 leaked fd's with MAGIC_COMPRESS,
  this has been fixed as of file-5.03

Replace equal/not equal uses of str[n]cmp() with rstreq[n] in python code

Replace equal/not equal uses of str[n]cmp() with rstreq[n] in binaries

Replace equal/not equal uses of str[n]cmp() with rstreq[n] in main ts code

Use rpmFileHasSuffix() instead of doing the same manually