Nullptr dereference in Bot::ChangeWptIndex() about yapb

[jeefo]

Well, its possible ofcourse, but GetPath could return nullptr only if requested index is out of bounds, but bounds are checked in ChangeWptIndex, or malloc has failed inside waypoint loader (but it's should throw exception).

Another case - broken waypoint file. What map?

[a1batross]

Unfortunately, I have only crash log.

*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
Build fingerprint: 'samsung/a5xeltextc/a5xeltextc:6.0.1/MMB29K/A510YDXS3BPK1:user/release-keys'
Revision: '3'
ABI: 'arm'
pid: 16881, tid: 16918, name: EngineThread >>> in.celest.xash3d.hl <<<
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x4
r0 00000000 r1 da118b84 r2 fffffcd0 r3 fffffcd4
r4 cd604800 r5 da117ef0 r6 da15b000 r7 da118b84
r8 00000000 r9 95a17d9a sl ffffffff fp da118b84
ip 00000000 sp dc8ff240 lr da0e5673 pc da0e5676 cpsr 88070030

backtrace:
#00 pc 00045676 /data/app/in.celest.xash3d.cs16client-2/lib/arm/libyapb_hardfp.so (_ZN3Bot14ChangeWptIndexEi+97)