lib/util.c: add assertions everywhere.
This commit is contained in:
parent
6010a24de6
commit
460515bf41
26
lib/util.c
26
lib/util.c
@ -130,6 +130,8 @@ xbps_pkg_version(const char *pkg)
|
|||||||
const char *p, *r;
|
const char *p, *r;
|
||||||
size_t p_len;
|
size_t p_len;
|
||||||
|
|
||||||
|
assert(pkg);
|
||||||
|
|
||||||
if ((p = strrchr(pkg, '-')) == NULL)
|
if ((p = strrchr(pkg, '-')) == NULL)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
@ -155,6 +157,8 @@ xbps_binpkg_pkgver(const char *pkg)
|
|||||||
char *p, *p1, *res;
|
char *p, *p1, *res;
|
||||||
unsigned int len;
|
unsigned int len;
|
||||||
|
|
||||||
|
assert(pkg);
|
||||||
|
|
||||||
/* skip path if found, only interested in filename */
|
/* skip path if found, only interested in filename */
|
||||||
if ((fname = strrchr(pkg, '/')))
|
if ((fname = strrchr(pkg, '/')))
|
||||||
fname++;
|
fname++;
|
||||||
@ -195,6 +199,8 @@ xbps_binpkg_arch(const char *pkg)
|
|||||||
char *p, *p1, *res;
|
char *p, *p1, *res;
|
||||||
unsigned int len;
|
unsigned int len;
|
||||||
|
|
||||||
|
assert(pkg);
|
||||||
|
|
||||||
/* skip path if found, only interested in filename */
|
/* skip path if found, only interested in filename */
|
||||||
if ((fname = strrchr(pkg, '/')))
|
if ((fname = strrchr(pkg, '/')))
|
||||||
fname++;
|
fname++;
|
||||||
@ -227,6 +233,8 @@ xbps_pkg_revision(const char *pkg)
|
|||||||
const char *p, *r;
|
const char *p, *r;
|
||||||
size_t p_len;
|
size_t p_len;
|
||||||
|
|
||||||
|
assert(pkg);
|
||||||
|
|
||||||
if ((p = strrchr(pkg, '-')) == NULL)
|
if ((p = strrchr(pkg, '-')) == NULL)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
@ -428,6 +436,9 @@ xbps_pkg_arch_match(struct xbps_handle *xhp, const char *orig,
|
|||||||
{
|
{
|
||||||
const char *arch;
|
const char *arch;
|
||||||
|
|
||||||
|
assert(xhp);
|
||||||
|
assert(orig);
|
||||||
|
|
||||||
if (xhp->target_arch)
|
if (xhp->target_arch)
|
||||||
arch = xhp->target_arch;
|
arch = xhp->target_arch;
|
||||||
else
|
else
|
||||||
@ -504,12 +515,18 @@ xbps_humanize_number(char *buf, int64_t bytes)
|
|||||||
size_t
|
size_t
|
||||||
xbps_strlcat(char *dest, const char *src, size_t siz)
|
xbps_strlcat(char *dest, const char *src, size_t siz)
|
||||||
{
|
{
|
||||||
|
assert(dest);
|
||||||
|
assert(src);
|
||||||
|
|
||||||
return strlcat(dest, src, siz);
|
return strlcat(dest, src, siz);
|
||||||
}
|
}
|
||||||
|
|
||||||
size_t
|
size_t
|
||||||
xbps_strlcpy(char *dest, const char *src, size_t siz)
|
xbps_strlcpy(char *dest, const char *src, size_t siz)
|
||||||
{
|
{
|
||||||
|
assert(dest);
|
||||||
|
assert(src);
|
||||||
|
|
||||||
return strlcpy(dest, src, siz);
|
return strlcpy(dest, src, siz);
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -524,6 +541,9 @@ xbps_pkg_reverts(xbps_dictionary_t pkg, const char *pkgver)
|
|||||||
const char *version = xbps_pkg_version(pkgver);
|
const char *version = xbps_pkg_version(pkgver);
|
||||||
const char *revertver = NULL;
|
const char *revertver = NULL;
|
||||||
|
|
||||||
|
assert(pkg);
|
||||||
|
assert(pkgver);
|
||||||
|
|
||||||
if ((reverts = xbps_dictionary_get(pkg, "reverts")) == NULL)
|
if ((reverts = xbps_dictionary_get(pkg, "reverts")) == NULL)
|
||||||
return false;
|
return false;
|
||||||
|
|
||||||
@ -572,6 +592,10 @@ xbps_symlink_target(struct xbps_handle *xhp, const char *path, const char *tgt)
|
|||||||
char *rootdir = NULL;
|
char *rootdir = NULL;
|
||||||
ssize_t r;
|
ssize_t r;
|
||||||
|
|
||||||
|
assert(xhp);
|
||||||
|
assert(path);
|
||||||
|
assert(tgt);
|
||||||
|
|
||||||
if (lstat(path, &sb) == -1)
|
if (lstat(path, &sb) == -1)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
@ -655,6 +679,8 @@ xbps_patterns_match(xbps_array_t patterns, const char *path)
|
|||||||
{
|
{
|
||||||
bool match = false;
|
bool match = false;
|
||||||
|
|
||||||
|
assert(path);
|
||||||
|
|
||||||
if (patterns == NULL)
|
if (patterns == NULL)
|
||||||
return false;
|
return false;
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user