Oops...

..and for parseForLang()
- return rpmRC like the others
- avoid alloca()
- streamline exit paths

..and the same for parseForConfig()
- streamline exists, avoid alloca()

parseForAttr() tweaks
- streamline exit paths a bit
- replace alloca() with xmalloc()
- avoid messing q value while checking for white space
- return rpmRC like the others

Avoid alloca() in parseForVerify(), streamline exit paths a bit

Avoid alloca() in parseForDev()

Oops, simplified it a little bit much, fixing

Avoid alloca() in rpmGetFilesystemUsage()

Avoid alloca() in getFilesystemList()

Remove remaining alloca() from rpmdb.c

Simplify dirName handling in rpmdbFindByFile()

Drop va_copy check, it's already done by C99 checks

Require C99 capable compiler in configure
- also enable C99 mode if not on by default

Remove alloca() ussage from rpmcache.c

Remove unused alloca'd variable

Remove alloca() from build()

Eliminate pointless alloca() of hsa struct

Remove bogus/misleading return value of hsaInit() and hsaFini()
- they don't (de)allocate anything, just modify contents of hsa arg

Remove useless hsa != NULL checks
- hsa has been already dereferenced before it's even checked for NULL in
  both hsaInit() and hsaNext()
- they're just internal interfaces with limited number of callers, we know
  hsa can't be NULL

Remove unused variable

Eliminate pointless alloca() of entryInfo and indexEntry structs

Eliminate pointless alloca() of local DBT structs

Eliminate pointless alloca() of struct fileIndexEntry struct

Eliminate pointless alloca() of availableIndexEntry struct.

Eliminate pointless alloca()'s of local dirInfo struct

Eliminate pointless alloca() of entryInfo and indexEntry structs

Eliminate pointless alloca() of local DBT structs.

Revert bunch of alloca() -> xcalloc() changes in rpmdb routines
- reverts the following commits:
  0b0dcd114028e1e2a00870917cf07a27858a30b1
  71018d6d2d67b0e17c737e7a90cc6bb704e224ba
  f6134083c29c6cbb3be255fadae47a03d35b09f1

Replace alloca() + memset() with xcalloc() in runImmedTriggers()

Avoid alloca() in markReplacedFiles()

Reduce alloca() use in relocateFileList()
- dirindexes and filelists can be *huge*
- filenames and such still allocated on stack, fix later...

Replace alloca() + memset() with xcalloc() in skipFiles()

Avoid alloca() on links in printFileInfo()
- also avoid unnecessary nulling of perms, no point ..

Avoid alloca() in rpmInstall()

Simplify setPathDefault()
- use rpmGetPath() to get sanitized path, avoid alloca()

Avoid alloca() in rpmteColorDS()

Single point of exit for dateToTimet(), avoid alloca()

Simplify lookupPackage(), avoid alloca()

Remove outdated comment, add a todo reminder
- callbacks for pkgs without files have been faked for some time now..

Might as well compare against tag shortname now that we have it

Update translations for a change

Remove bogus const from rpmfi pre- and posttrans members
- they're always duped, scaremem or not

Eliminate rpmtsi refcount debug junk from the API

Update doxygen docs for rpmteNEVR() and friends

Remove meaningless extern on function declarations in headers

Add rpmteEVR() function
- seems like a useful thing to have
- use in ensureOlder()

I'm out of words for this one.

Check for chroot() result in rpmtsRun()
- bail out early if it fails instead of blindly continuing and potentially
  messing in real root (chroot can fail for priviledged user too)

Make verification with --root reliable (rhbz#434150)
- use chroot() instead of vain path manipulation games, otherwise we'll get
  bogus results from users and groups not matching etc
- non-priviledged verification is unreliable anyway but running rpm under
  fakechroot (unpriviledged) gives much better results than just prepending
  root to paths

Force preloading of name service libraries early in initialization
- this used to be done in psm chroot operation, but we can end up chrooting
  before that and without psm being involved
- don't bother with getpwnam() etc, a single gethostbyname() loads all
  that we need

Relax spec permissions in tar build mode (rhbz#253648)
- mkstemp() can create the spec in overly protective mode (rpmlint whines),
  relax up to what users umask permits

Simplify rebuild path generation

More path manipulation craziness eliminated in rpmdbRemoveDatabase()

Eliminate wtf dbpath shuffling in rpmdbMoveDatabase()
- Just use rpmGetPath() to construct a clean, sanitized path from the
  components.
- Previously it went to great lengths to ensure trailing / on dbpath,
  catenate the paths with %s/%s/%s and then rpmCleanPath() to get rid
  of the extra slashes just created. Eh.

Don't use alloca for struct stat, there's no need to.

Add rpmdb internal helper for cleaning up db4 environment, use it

Recognize lzma payload from its magic
- old lzma formats without magic aren't supported

Prefer lzma's default -7 compression

Main lzma rpmio part based on SuSE patches, thanks for them!

Handle lzma payload

Check for lzma library in configure.ac and reference it in Makefile.am

Allow lzma payload in writeRPM()

Rip i386-specific RDTSC support from rpmsw, use gettimeofday() everywhere
- we don't need accuracy beyond what gettimeofday() offers for bleeping
  debugging benchmarks
- we especially don't need hw-specific magic asm voodoo to get unreliable
  timing results (RDTSC isn't reliable with multi-core/hyperthreaded CPU's
  etc etc)

Avoid %patch0 getting applied twice

Retrofit RPMTAG_SOURCEPACKAGE to source rpms.

Use argvAppend() instead of manual reallocs, plug a leak from rpmGlob()

Use argvNew() instead of manual xcalloc()

Allow setting fuzz factor  to 0
- use --fuzz=num instead of -Fnum or -F num to be sure
  /usr/bin/patch won't complain

Don't use static buffers

Fail if it is impossible to chdir to directory to run helper script in

Fix find-requires (rhbz#443015), thanks to Bill Nottingham

Sanitize + simplify %patch "macro" parsing
- Use popt for parsing %patch options, give accurate error messages for
  invalid options instead of bogosity like "bad %patch arg: %build"
- Store all the variants we can get patch numbers into argv to avoid
  special cases when passing to doPatch(), this has the added bonus
  of removing the 1024 patches per line-limit (as if somebody cared)

Single exit point for doPatchMacro()
- assuming failure avoids several redundant free() + return RPMRC_FAILS

Remove ugly hardcoded docDirs list, use a macro instead
- %__docdir_path contains a colon-delimited list of documentation dirs,
  macros expanded before use
- dynamically allocate docDirs as needed using an argv array, removes
  the hardcoded MAXDOCDIRS limit on docdirs (not that it matters much)

Avoid entirely silly rpmio FD_t use in processPackageFiles()
- since we just convert it to FILE anyway, no much point in using
  Fopen() then
- additionally remember to close the file if expandMacros() fails

Change rpmGlob() argvPtr type to ARGV_t to make it obvious what it is
- also fix incorrect doxygen commentary, it wasn't a contiguous blob
  before either

Use argvFree() instead of manual free in loop

Further simplify rpmInitMacros()
- Use argvSplit() for splitting the macro path to components instead of
  manual pointer-parsery.
- If URL's are to be supported or accepted at all (previous code attempted
  to skip them), ':' is a very poor delimiter character.

Use argvFree() to free, well, an argv, instead of manual freeing

Simplify rpmGlob() a bit
- use argv*() routines instead of manual argv manipulation

Dynamically allocate buffers in rpmGetPath()
- avoid unbounded copying to static sized buffer
- use rpmExpand() instead of expandMacros() which requires preallocated
  buffer when we've no idea of expanded size

Avoid entirely silly rpmio FD_t use
- rdcl() converts to FILE internally anyway, might as well use fopen()
  to begin with...

Remove unnecessary xstrdup() of dynamically allocated buffer
- leftover from when target buffer was static...

Cosmetics: group operations more logically...

Simplify macro option + argument processing
- eliminate ninja-voodoo pointer-hackery parsing, replace argvSplit() +
  argvJoin() and friends
- eliminate static sized buffers from the processing

Add argvJoin() for combining an argv array into a string

Put '\0' to the end of xmalloc'd string to avoid surprises

Eliminate splitString() and freeSplitString()
- use ARGV_t instead, it does the same and more...

Accidental typo introduced to rpmrc.c
- seems to be one of those days, sigh

Fix dumb off-by-one errors introduced by rstrlcpy() changes

Bunch of strncpy/strcpy -> rstrlcpy() converts in rpmrc

Add missing include

Use rstrlcpy() for various parseForFoo() functions in build
- just avoids manual termination, no new fixes here..

Use rstrlcpy() for parseRCPOT() version + release
- just avoids manual zero-termination

Use rstrlcpy() for printFileInfo() username + group fields
- just avoids having to manually ensure zero-termination

Make sure lead->name is zero-terminated

Add rstrlcpy() string helper function
- adopted from strlcpy() function of OpenBSD originally developed by
  Todd C. Miller <Todd.Miller@courtesan.com>

More accurate filtering of .rpmnew etc when reading macros
- strstr() can give false positives, use rpmFileHasSuffix() instead

Have gcc check for NULL terminated arglist in rstrscat

Use sizeof() instead of direct element count in rpmluaGetVar()