proto-ext-whiterabbit/wr-servo: fix the overflow of casted picos_mu

For long liks (like 180km) "(signed)s->picos_mu" overflows causing wrong result of the comparison: (signed)s->picos_mu < (signed)big_delta_fix) Signed-off-by: Adam Wujek <adam.wujek@cern.ch>

proto-ext-whiterabbit/wr-servo: fix the overflow of casted picos_mu
For long liks (like 180km) "(signed)s->picos_mu" overflows causing wrong result of the comparison: (signed)s->picos_mu < (signed)big_delta_fix) Signed-off-by: Adam Wujek <adam.wujek@cern.ch>
8cead1c6 · Adam Wujek · 04626f4a · 8cead1c6
Commit 8cead1c6 authored Mar 31, 2016 by Adam Wujek 💬
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

wr-servo.c proto-ext-whiterabbit/wr-servo.c +1 -1

No files found.
--- a/proto-ext-whiterabbit/wr-servo.c
+++ b/proto-ext-whiterabbit/wr-servo.c
@@ -276,7 +276,7 @@ int wr_servo_update(struct pp_instance *ppi)
 	big_delta_fix =  s->delta_tx_m + s->delta_tx_s
 		       + s->delta_rx_m + s->delta_rx_s;

-	if ((signed)s->picos_mu < (signed)big_delta_fix) {
+	if ((int64_t)s->picos_mu < (int64_t)big_delta_fix) {
 		/* avoid negatives in calculations */
 		s->picos_mu = big_delta_fix;
 	}