Fix refcounting bug in Pair::at()
commitda7a1b61df154fdb21f883f575e1ccae4fadcb46
authorDrew Paroski <andrewparoski@fb.com>
Tue, 3 Jun 2014 08:13:59 +0000 (3 01:13 -0700)
committerJoelMarcey <joelm@fb.com>
Tue, 3 Jun 2014 23:58:57 +0000 (3 16:58 -0700)
tree56411fef04b054ba47b56e3ca984612df4710044
parentca47baca450dece0597e63330ad27617bf1feeb6
Fix refcounting bug in Pair::at()

Summary: Pair::at() was not incReffing the value it returned, causing HHVM to free
the value early causing crashes or other bad behavior. This diff fixes the
refcounting bug.

Reviewed By: @elgenie

Differential Revision: D1361339
hphp/runtime/ext/ext_collections.cpp
hphp/test/slow/collection_classes/823.php
hphp/test/slow/collection_classes/823.php.expect