Igen, ez elég nagy hülyeség. Az std::string megvalósításától függ, hogy mi van benne, arról meg nem tételezhetsz fel semmit. Ennek a kódrészletnek akármi (fals negatív, fals pozitív és akár programfagyi is) lehet az eredménye.
Szerk: A post megírása után felébredtem, persze először mást láttam bele, bár a lényegen nem változtat ;-)
A legnagyobb gond az, hogy a tömb utáni memóriaterületet próbálod stringként felhasználni, pedig lehet ott bármi. A másik gond az, hogy a string tartalmában keresel 0-31-ig karaktereket, amit jó eséllyel soha nem fogsz találni.