Merge pull request #25470 from keszybz/strv-extendf-format
Improve check for strv_extendf() format string
This commit is contained in:
+1
-1
@@ -45,7 +45,7 @@ static inline int strv_extend(char ***l, const char *value) {
|
||||
return strv_extend_with_size(l, NULL, value);
|
||||
}
|
||||
|
||||
int strv_extendf(char ***l, const char *format, ...) _printf_(2,0);
|
||||
int strv_extendf(char ***l, const char *format, ...) _printf_(2,3);
|
||||
int strv_extend_front(char ***l, const char *value);
|
||||
|
||||
int strv_push_with_size(char ***l, size_t *n, char *value);
|
||||
|
||||
+4
-4
@@ -948,11 +948,11 @@ static int activation_details_timer_append_env(ActivationDetails *details, char
|
||||
if (!dual_timestamp_is_set(&t->last_trigger))
|
||||
return 0;
|
||||
|
||||
r = strv_extendf(strv, "TRIGGER_TIMER_REALTIME_USEC=%" USEC_FMT, t->last_trigger.realtime);
|
||||
r = strv_extendf(strv, "TRIGGER_TIMER_REALTIME_USEC=" USEC_FMT, t->last_trigger.realtime);
|
||||
if (r < 0)
|
||||
return r;
|
||||
|
||||
r = strv_extendf(strv, "TRIGGER_TIMER_MONOTONIC_USEC=%" USEC_FMT, t->last_trigger.monotonic);
|
||||
r = strv_extendf(strv, "TRIGGER_TIMER_MONOTONIC_USEC=" USEC_FMT, t->last_trigger.monotonic);
|
||||
if (r < 0)
|
||||
return r;
|
||||
|
||||
@@ -974,7 +974,7 @@ static int activation_details_timer_append_pair(ActivationDetails *details, char
|
||||
if (r < 0)
|
||||
return r;
|
||||
|
||||
r = strv_extendf(strv, "%" USEC_FMT, t->last_trigger.realtime);
|
||||
r = strv_extendf(strv, USEC_FMT, t->last_trigger.realtime);
|
||||
if (r < 0)
|
||||
return r;
|
||||
|
||||
@@ -982,7 +982,7 @@ static int activation_details_timer_append_pair(ActivationDetails *details, char
|
||||
if (r < 0)
|
||||
return r;
|
||||
|
||||
r = strv_extendf(strv, "%" USEC_FMT, t->last_trigger.monotonic);
|
||||
r = strv_extendf(strv, USEC_FMT, t->last_trigger.monotonic);
|
||||
if (r < 0)
|
||||
return r;
|
||||
|
||||
|
||||
Reference in New Issue
Block a user