I really wish these things were caught at compile time.  I've screwed stuff up like this a few times.

-- Vossel

----- Original Message -----
From: "Andrew Beekhof" <notifications@github.com>
To: "ClusterLabs/pacemaker" <pacemaker@noreply.github.com>
Cc: "David Vossel" <dvossel@redhat.com>
Sent: Thursday, August 22, 2013 12:32:38 AM
Subject: Re: [pacemaker] Fix: Invalid formatting of log message causes crash (#341)

> @@ -367,7 +367,7 @@ static gboolean lrm_state_verify_stopped(lrm_state_t * lrm_state, enum crmd_fsa_
>      }
>  
>      if (counter > 0) {
> -        do_crm_log(log_level, "%d pending LRM operations at %s%s", counter, when);

Good catch

---
Reply to this email directly or view it on GitHub:
https://github.com/ClusterLabs/pacemaker/pull/341/files#r5916871