CLEANUP: quic: Rename <buf> variable into quic_padding_check()

Make quic_padding_check() be more readable: there is not struct buffer variable
used by this function.

Should be backported to 2.7.

diff --git a/src/quic_conn.c b/src/quic_conn.c
index 399f22f349427c3fa3ffeda29ef5073d72e6f32f..6f0827525da260d0d4a94d136753aed599bde640 100644 (file)
--- a/src/quic_conn.c
+++ b/src/quic_conn.c
@@ -6749,18 +6749,18 @@ static int qc_conn_alloc_ssl_ctx(struct quic_conn *qc)
        goto leave;
 }
 
-/* Check that all the bytes between <buf> included and <end> address
+/* Check that all the bytes between <pos> included and <end> address
  * excluded are null. This is the responsibility of the caller to
- * check that there is at least one byte between <buf> end <end>.
+ * check that there is at least one byte between <pos> end <end>.
  * Return 1 if this all the bytes are null, 0 if not.
  */
-static inline int quic_padding_check(const unsigned char *buf,
+static inline int quic_padding_check(const unsigned char *pos,
                                      const unsigned char *end)
 {
-       while (buf < end && !*buf)
-               buf++;
+       while (pos < end && !*pos)
+               pos++;
 
-       return buf == end;
+       return pos == end;
 }
 
 /* Find the associated connection to the packet <pkt> or create a new one if