The find_builtin_prop internal routine in dispex.c implements a binary
search with the requirement that a class properties array be in
ascending order, so this fix allows String's properties array to
satisfy this requirement and allows the tests for String.sub() to
pass.

As pointed out by Dan Kegel.

This is consistent with other uses of struct color_fixup_desc.

This fixes a regression introduced by commit
89139b75.

crypt32: Allow setting CERT_KEY_PROV_HANDLE_PROP_ID if CERT_KEY_CONTEXT_PROP_ID has not previously been set.

Don't allow fstat() to fail for no reason.
Better check the st_mode field.