pkt_sched: Stop using NLA_PUT*().

These macros contain a hidden goto, and are thus extremely error
prone and make code hard to audit.

Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 14 insertions, 7 deletions

diff --git a/net/sched/sch_netem.c b/net/sched/sch_netem.c
index 5da548fa7ae..110973145a4 100644
--- a/net/sched/sch_netem.c
+++ b/net/sched/sch_netem.c
@@ -834,7 +834,8 @@ static int dump_loss_model(const struct netem_sched_data *q,
                         .p23 = q->clg.a5,
                 };
 
-                NLA_PUT(skb, NETEM_LOSS_GI, sizeof(gi), &gi);
+                if (nla_put(skb, NETEM_LOSS_GI, sizeof(gi), &gi))
+                        goto nla_put_failure;
                 break;
         }
         case CLG_GILB_ELL: {
@@ -845,7 +846,8 @@ static int dump_loss_model(const struct netem_sched_data *q,
                         .k1 = q->clg.a4,
                 };
 
-                NLA_PUT(skb, NETEM_LOSS_GE, sizeof(ge), &ge);
+                if (nla_put(skb, NETEM_LOSS_GE, sizeof(ge), &ge))
+                        goto nla_put_failure;
                 break;
         }
         }
@@ -874,26 +876,31 @@ static int netem_dump(struct Qdisc *sch, struct sk_buff *skb)
         qopt.loss = q->loss;
         qopt.gap = q->gap;
         qopt.duplicate = q->duplicate;
-        NLA_PUT(skb, TCA_OPTIONS, sizeof(qopt), &qopt);
+        if (nla_put(skb, TCA_OPTIONS, sizeof(qopt), &qopt))
+                goto nla_put_failure;
 
         cor.delay_corr = q->delay_cor.rho;
         cor.loss_corr = q->loss_cor.rho;
         cor.dup_corr = q->dup_cor.rho;
-        NLA_PUT(skb, TCA_NETEM_CORR, sizeof(cor), &cor);
+        if (nla_put(skb, TCA_NETEM_CORR, sizeof(cor), &cor))
+                goto nla_put_failure;
 
         reorder.probability = q->reorder;
         reorder.correlation = q->reorder_cor.rho;
-        NLA_PUT(skb, TCA_NETEM_REORDER, sizeof(reorder), &reorder);
+        if (nla_put(skb, TCA_NETEM_REORDER, sizeof(reorder), &reorder))
+                goto nla_put_failure;
 
         corrupt.probability = q->corrupt;
         corrupt.correlation = q->corrupt_cor.rho;
-        NLA_PUT(skb, TCA_NETEM_CORRUPT, sizeof(corrupt), &corrupt);
+        if (nla_put(skb, TCA_NETEM_CORRUPT, sizeof(corrupt), &corrupt))
+                goto nla_put_failure;
 
         rate.rate = q->rate;
         rate.packet_overhead = q->packet_overhead;
         rate.cell_size = q->cell_size;
         rate.cell_overhead = q->cell_overhead;
-        NLA_PUT(skb, TCA_NETEM_RATE, sizeof(rate), &rate);
+        if (nla_put(skb, TCA_NETEM_RATE, sizeof(rate), &rate))
+                goto nla_put_failure;
 
         if (dump_loss_model(q, skb) != 0)
                 goto nla_put_failure;