fb_compact_unserialize: raise notice for invalid vector
commit1849ab95feaf28b8ad4e8db5eeacd328043408d3
authorBill Fumerola <bill@fb.com>
Fri, 14 Sep 2018 14:40:42 +0000 (14 07:40 -0700)
committerHhvm Bot <hhvm-bot@users.noreply.github.com>
Fri, 14 Sep 2018 14:42:31 +0000 (14 07:42 -0700)
tree166372659815d1fe97759a9ebda817b7df88517c
parent6e7694534768b2474700837c8813a006db8de774
fb_compact_unserialize: raise notice for invalid vector

Summary:
add a gated runtime option to complain about any compacted data which contains an invalid skip code inside a vector.
the "specification" docblock does not state this is valid, but our implementation allows it.
hhvm does not produce these vector codes, so the only implementation that could have used skip inside a vector is outside implementations.

Reviewed By: ricklavoie

Differential Revision: D9646688

fbshipit-source-id: c1ec2df71c77c598d1b871a8677fb152fd416861
hphp/compiler/option.cpp
hphp/runtime/base/runtime-option.h
hphp/runtime/ext/fb/ext_fb.cpp
hphp/test/slow/ext_fb/compact_serialize.php
hphp/test/slow/ext_fb/compact_serialize.php.expect
hphp/test/slow/ext_fb/compact_serialize.php.hphp_opts [new file with mode: 0644]
hphp/test/slow/ext_fb/compact_serialize.php.opts [new file with mode: 0644]