vpx_timestamp,gcd: assert params are positive
authorJames Zern <jzern@google.com>
Sat, 18 Jan 2020 07:33:35 +0000 (23:33 -0800)
committerJames Zern <jzern@google.com>
Wed, 22 Jan 2020 00:12:25 +0000 (16:12 -0800)
this function is currently only used with range checked timestamp
values, but this documents the function's expectations in case it's used
elsewhere

Change-Id: I9de314fc500a49f34f8a1df3598d64bc5070248e

vpx_util/vpx_timestamp.h

index c210de5..5296458 100644 (file)
@@ -11,6 +11,8 @@
 #ifndef VPX_VPX_UTIL_VPX_TIMESTAMP_H_
 #define VPX_VPX_UTIL_VPX_TIMESTAMP_H_
 
+#include <assert.h>
+
 #ifdef __cplusplus
 extern "C" {
 #endif  // __cplusplus
@@ -23,7 +25,9 @@ typedef struct vpx_rational64 {
 
 static INLINE int gcd(int64_t a, int b) {
   int r;  // remainder
-  while (b > 0) {
+  assert(a >= 0);
+  assert(b > 0);
+  while (b != 0) {
     r = (int)(a % b);
     a = b;
     b = r;