Iterator's value is not checked properly in range_run class.

[g.gupta@…]

In File boost/spirit/home/classic/utility/impl/chset/range_run.ipp

if (iter != run.end() && iter->includes(v))
                    return true;
                if (iter != run.begin())
                    return (--iter)->includes(v);
            }
            return false;

Here iter = run.end() condition is not checked.
So, attached patch is the fix for it.

[g.gupta@…]

Patch file for the reported Bug.

[Joel de Guzman]

Do you have a minimal test case exhibiting this edge case problem? I'll need one. I think the logic is just fine as it is. If iter is at the end, it gets decremented before dereferencing it (but only if it's not at begin already).

[Nikita Kniazev <nok.raven@…>]

There is no problem to decrement an end iterator, C++ Standard allows it. So it is preferably valid code and I see no reason to check add a check for end iter != run.end() before decrementing --iter.