holland lvm does not check for invalid snapshots correctly

[Andrew Garner]

When holland lvm fails due to an invalid snapshot, there is logic to detect this and provide a helpful error message on what happened. However, holland is only checking for an 'S' lv_attr - which is questionable logic by itself. On RHEL6 (and probably rhel5.7+ at least), invalid snapshot should have the "I" ((I)nvalid snapshot) generally rather than the "S" (invalid (S)uspended snapshot) flag.

Additionally holland should be specifically checking the 5th lv_attr bit rather than searching anywhere in the lv_attr string.