chiark / gitweb /
sd-rtnl: use GREEDY_REALLOC for message queues
authorTom Gundersen <teg@jklm.no>
Thu, 10 Apr 2014 17:00:30 +0000 (19:00 +0200)
committerTom Gundersen <teg@jklm.no>
Thu, 10 Apr 2014 17:00:36 +0000 (19:00 +0200)
src/libsystemd/sd-rtnl/rtnl-internal.h
src/libsystemd/sd-rtnl/sd-rtnl.c

index 8aa2300835d54e706ff457b2b8dfc3cd5ef038d3..9003dad312c09b7b37191afe24abe9681732a54f 100644 (file)
@@ -66,9 +66,11 @@ struct sd_rtnl {
 
         sd_rtnl_message **rqueue;
         unsigned rqueue_size;
+        size_t rqueue_allocated;
 
         sd_rtnl_message **wqueue;
         unsigned wqueue_size;
+        size_t wqueue_allocated;
 
         bool processing:1;
 
index 2ab9d90aa790d4f76360d6adc45111d2b6a6fe1b..8e709bb66dfb16c3a9c73a7bf47af178b1e9006a 100644 (file)
@@ -51,8 +51,7 @@ static int sd_rtnl_new(sd_rtnl **ret) {
 
         /* We guarantee that wqueue always has space for at least
          * one entry */
-        rtnl->wqueue = new(sd_rtnl_message*, 1);
-        if (!rtnl->wqueue)
+        if (!GREEDY_REALLOC(rtnl->wqueue, rtnl->wqueue_allocated, 1))
                 return -ENOMEM;
 
         *ret = rtnl;
@@ -188,18 +187,14 @@ int sd_rtnl_send(sd_rtnl *nl,
                         nl->wqueue_size = 1;
                 }
         } else {
-                sd_rtnl_message **q;
-
                 /* append to queue */
                 if (nl->wqueue_size >= RTNL_WQUEUE_MAX)
                         return -ENOBUFS;
 
-                q = realloc(nl->wqueue, sizeof(sd_rtnl_message*) * (nl->wqueue_size + 1));
-                if (!q)
+                if (!GREEDY_REALLOC(nl->wqueue, nl->wqueue_allocated, nl->wqueue_size + 1))
                         return -ENOMEM;
 
-                nl->wqueue = q;
-                q[nl->wqueue_size ++] = sd_rtnl_message_ref(message);
+                nl->wqueue[nl->wqueue_size ++] = sd_rtnl_message_ref(message);
         }
 
         if (serial)