all: rework how arrays work

Instead of having a null pointer, use a dummy array which just
says the array is empty. This helps in multiple places of the code
which would otherwise need explicitly need to check first if the
array exists. This has been cause of multiple seg.faults in the
past as the array check is easily omitted.

This also removes (or fixes) all existing checks accordingly.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/fix.c b/src/fix.c
index f84fbc8..8b9021a 100644
--- a/src/fix.c
+++ b/src/fix.c
@@ -69,7 +69,7 @@ static int fix_main(void *pctx, struct apk_database *db, int argc, char **argv)
 			apk_dep_from_blob(&deps[i], db, APK_BLOB_STR(argv[i]));
 			name = deps[i].name;
 
-			for (j = 0; name->pkgs && j < name->pkgs->num; j++) {
+			for (j = 0; j < name->pkgs->num; j++) {
 				if (name->pkgs->item[j]->ipkg != NULL) {
 					pkg = name->pkgs->item[j];
 					break;