diff options
author | Eli Schwartz <eschwartz@archlinux.org> | 2018-06-13 16:34:32 +0200 |
---|---|---|
committer | Allan McRae <allan@archlinux.org> | 2018-07-27 03:43:39 +0200 |
commit | 1a5f308d52f3a81990e9992400d1de7d449ef951 (patch) | |
tree | 00d4beea60c28886165e6e6b895a0aa040ede3f6 /lib/libalpm/md5driver.c | |
parent | 2d8a751943abfe6c32fdc6957d61848d5eb0e4ca (diff) | |
download | pacman-1a5f308d52f3a81990e9992400d1de7d449ef951.tar.gz pacman-1a5f308d52f3a81990e9992400d1de7d449ef951.tar.xz |
makepkg: optimize and fix BUILDINFO generation's use of awk
The biggest issue is directly supplying the data within the format
string which can result in misinterpreting formatter sequences if a
printed variable contains an "%" in it. This character is currently
permitted in the pkgver field, though not in the pkgname. Also
pacman/libalpm itself has much looser limitations and this can appear
anywhere at all if a package was created by some other program.
For the package "iambroke-1%s-1-any.pkg.tar.xz", installed in the build
environment, the result is:
-> Generating .BUILDINFO file...
awk: cmd. line:3: (FILENAME=- FNR=1085) fatal: not enough arguments to satisfy format string
`-1%s-1'
^ ran out for this one
Followed by a .BUILDINFO which contains an LC_ALL=C sorted list of
$pkgname-${epoch:+$epoch:}$pkgver-$pkgrel-$arch ending in:
installed = iambroke
Which is cut short, then fails to list the succeeding packages. The
package itself successfully builds.
It's also unnecessary to save the output of pacman -Qq in order to get the
information for pacman -Qi, since the latter will, just like the former,
return information for all installed packages if not given a package
name(s).
While I am at it, pipe this directly to awk rather than keeping a copy
in an unnecessary local variable. This is slightly more efficient in
addition to preventing the <<< herestring from re-interpreting the
content of "$pkginfos" in ways that don't really matter for our usage.
Signed-off-by: Eli Schwartz <eschwartz@archlinux.org>
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'lib/libalpm/md5driver.c')
0 files changed, 0 insertions, 0 deletions