Andreas Gampe suggested a different fix.

(Bionic _does_ do the malloc(0)!=0 thing, the problem is pedantic access
checking, not null dereference. So test count instead.)

1 files changed, 2 insertions, 1 deletions

diff --git a/toys/posix/ps.c b/toys/posix/ps.c
index 8ee68ad6..e94902b3 100644
--- a/toys/posix/ps.c
+++ b/toys/posix/ps.c
@@ -1417,7 +1417,8 @@ static void top_common(
     mix.count = 0;
 
     while (old.count || new.count) {
-      struct carveup *otb = old.tb ? *old.tb : 0, *ntb = new.tb ? *new.tb : 0;
+      struct carveup *otb = old.count ? *old.tb : 0,
+                     *ntb = new.count ? *new.tb : 0;
 
       // If we just have old for this process, it exited. Discard it.
       if (old.count && (!new.count || *otb->slot < *ntb->slot)) {