[Pacemaker] crm_failcount would take some irrelevant values

[Junko IKEDA]

> but 0 is a valid value - and possibly the most likely one to be set
> and that strcmp looks wrong to me
> 
> 
> i'll apply a modified version of this, but i don't want to get too
> hung up trying to detect every possible value a hostile admin could
> throw at the cluster (especially when they can just as easily use
> cibadmin to achieve the same result).
> 
> for 1.0 we'll be using relax-NG schemas (instead of DTDs) for
> validation which will allow us to catch more things like this
> centrally (on the server) and reject them (and defining what is a
> valid 'score' is one thing I've already done :-).

Ok, I understand that this behavior would be better in 1.0.
We have to check RELAX NG for now.
Thanks!

Thanks,
Junko