<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>28779</bug_id>
          
          <creation_ts>2009-08-27 12:07:28 -0700</creation_ts>
          <short_desc>JSON parser accepts trailing comma in array literals</short_desc>
          <delta_ts>2009-08-27 12:15:52 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>JavaScriptCore</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Oliver Hunt">oliver</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>143161</commentid>
    <comment_count>0</comment_count>
    <who name="Oliver Hunt">oliver</who>
    <bug_when>2009-08-27 12:07:28 -0700</bug_when>
    <thetext>JSON.parse(&quot;[1,]&quot;) succeeds but is invalid per the JSON spec.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>143163</commentid>
    <comment_count>1</comment_count>
      <attachid>38677</attachid>
    <who name="Oliver Hunt">oliver</who>
    <bug_when>2009-08-27 12:10:43 -0700</bug_when>
    <thetext>Created attachment 38677
Patch v1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>143164</commentid>
    <comment_count>2</comment_count>
      <attachid>38677</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2009-08-27 12:13:24 -0700</bug_when>
    <thetext>Comment on attachment 38677
Patch v1

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>143166</commentid>
    <comment_count>3</comment_count>
    <who name="Oliver Hunt">oliver</who>
    <bug_when>2009-08-27 12:15:52 -0700</bug_when>
    <thetext>Committed r47828</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>38677</attachid>
            <date>2009-08-27 12:10:43 -0700</date>
            <delta_ts>2009-08-27 12:13:24 -0700</delta_ts>
            <desc>Patch v1</desc>
            <filename>bug-28779-20090827121042.patch</filename>
            <type>text/plain</type>
            <size>3215</size>
            <attacher name="Oliver Hunt">oliver</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZyBiL0phdmFTY3JpcHRDb3JlL0No
YW5nZUxvZwppbmRleCAxNTBkMGFmLi4yYTlkODJkIDEwMDY0NAotLS0gYS9KYXZhU2NyaXB0Q29y
ZS9DaGFuZ2VMb2cKKysrIGIvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUg
QEAKKzIwMDktMDgtMjcgIE9saXZlciBIdW50ICA8b2xpdmVyQGFwcGxlLmNvbT4KKworICAgICAg
ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBKU09OIHBhcnNlciBhY2Nl
cHRzIHRyYWlsaW5nIGNvbW1hIGluIGFycmF5IGxpdGVyYWxzCisgICAgICAgIGh0dHBzOi8vYnVn
cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yODc3OQorCisgICAgICAgIFVwZGF0ZSBwYXJz
ZXIgdG8gY29ycmVjdGx5IGZhaWwgaWYgdGhlcmUncyBhIHRyYWlsaW5nIGNvbW1hLgorCisgICAg
ICAgICogcnVudGltZS9MaXRlcmFsUGFyc2VyLmNwcDoKKyAgICAgICAgKEpTQzo6TGl0ZXJhbFBh
cnNlcjo6cGFyc2UpOgorCiAyMDA5LTA4LTI2ICBPbGl2ZXIgSHVudCAgPG9saXZlckBhcHBsZS5j
b20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgR2F2aW4gQmFycmFjbG91Z2guCmRpZmYgLS1naXQg
YS9KYXZhU2NyaXB0Q29yZS9ydW50aW1lL0xpdGVyYWxQYXJzZXIuY3BwIGIvSmF2YVNjcmlwdENv
cmUvcnVudGltZS9MaXRlcmFsUGFyc2VyLmNwcAppbmRleCAxN2VjOTA2Li5kMjQyMjgyIDEwMDY0
NAotLS0gYS9KYXZhU2NyaXB0Q29yZS9ydW50aW1lL0xpdGVyYWxQYXJzZXIuY3BwCisrKyBiL0ph
dmFTY3JpcHRDb3JlL3J1bnRpbWUvTGl0ZXJhbFBhcnNlci5jcHAKQEAgLTI5NSw3ICsyOTUsMTAg
QEAgSlNWYWx1ZSBMaXRlcmFsUGFyc2VyOjpwYXJzZShQYXJzZXJTdGF0ZSBpbml0aWFsU3RhdGUp
CiAgICAgICAgICAgICB9CiAgICAgICAgICAgICBkb1BhcnNlQXJyYXlTdGFydEV4cHJlc3Npb246
CiAgICAgICAgICAgICBjYXNlIERvUGFyc2VBcnJheVN0YXJ0RXhwcmVzc2lvbjogeworICAgICAg
ICAgICAgICAgIFRva2VuVHlwZSBsYXN0VG9rZW4gPSBtX2xleGVyLmN1cnJlbnRUb2tlbigpLnR5
cGU7CiAgICAgICAgICAgICAgICAgaWYgKG1fbGV4ZXIubmV4dCgpID09IFRva1JCcmFja2V0KSB7
CisgICAgICAgICAgICAgICAgICAgIGlmIChsYXN0VG9rZW4gPT0gVG9rQ29tbWEpCisgICAgICAg
ICAgICAgICAgICAgICAgICByZXR1cm4gSlNWYWx1ZSgpOwogICAgICAgICAgICAgICAgICAgICBt
X2xleGVyLm5leHQoKTsKICAgICAgICAgICAgICAgICAgICAgbGFzdFZhbHVlID0gb2JqZWN0U3Rh
Y2subGFzdCgpOwogICAgICAgICAgICAgICAgICAgICBvYmplY3RTdGFjay5yZW1vdmVMYXN0KCk7
CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cKaW5kZXggYzM5YmIyNS4uMjQ5MTI4MiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdl
TG9nCisrKyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDA5LTA4
LTI3ICBPbGl2ZXIgSHVudCAgPG9saXZlckBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgSlNPTiBwYXJzZXIgYWNjZXB0cyB0cmFpbGlu
ZyBjb21tYSBpbiBhcnJheSBsaXRlcmFscworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9Mjg3NzkKKworICAgICAgICBDb3JyZWN0IHRoZSBKU09OLnBhcnNl
IHRlc3QgdG8gZXhwZWN0ICJbMSxdIiB0byB0aHJvdy4KKworICAgICAgICAqIGZhc3QvanMvSlNP
Ti1wYXJzZS1leHBlY3RlZC50eHQ6CisgICAgICAgICogZmFzdC9qcy9yZXNvdXJjZXMvSlNPTi1w
YXJzZS5qczoKKyAgICAgICAgKGNyZWF0ZVRlc3RzLnJlc3VsdCk6CisKIDIwMDktMDgtMjYgIEFu
ZHJhcyBCZWNzaSAgPGJlY3NpLmFuZHJhc0BzdHVkLnUtc3plZ2VkLmh1PgogCiAgICAgICAgIFJl
dmlld2VkIGJ5IEFyaXlhIEhpZGF5YXQuCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9mYXN0L2pz
L0pTT04tcGFyc2UtZXhwZWN0ZWQudHh0IGIvTGF5b3V0VGVzdHMvZmFzdC9qcy9KU09OLXBhcnNl
LWV4cGVjdGVkLnR4dAppbmRleCBmMGYyYWY2Li44N2QwZDJhIDEwMDY0NAotLS0gYS9MYXlvdXRU
ZXN0cy9mYXN0L2pzL0pTT04tcGFyc2UtZXhwZWN0ZWQudHh0CisrKyBiL0xheW91dFRlc3RzL2Zh
c3QvanMvSlNPTi1wYXJzZS1leHBlY3RlZC50eHQKQEAgLTk0LDcgKzk0LDggQEAgUEFTUyBKU09O
LnN0cmluZ2lmeSh0ZXN0c1tpXShuYXRpdmVKU09OKSkgaXMgSlNPTi5zdHJpbmdpZnkodGVzdHNb
aV0oSlNPTikpCiBmdW5jdGlvbiAoanNvbk9iamVjdCkgewogICAgICAgICByZXR1cm4ganNvbk9i
amVjdC5wYXJzZSgnWzEsXScpOwogICAgIH0KLVBBU1MgSlNPTi5zdHJpbmdpZnkodGVzdHNbaV0o
bmF0aXZlSlNPTikpIGlzIEpTT04uc3RyaW5naWZ5KHRlc3RzW2ldKEpTT04pKQorUEFTUyB0ZXN0
c1tpXShuYXRpdmVKU09OKSB0aHJldyBleGNlcHRpb24gU3ludGF4RXJyb3I6IFVuYWJsZSB0byBw
YXJzZSBKU09OIHN0cmluZy4KK2pzb24yLmpzIGRpZCBub3QgdGhyb3cgZm9yIGEgdGVzdCB3ZSBl
eHBlY3QgdG8gdGhyb3cuCiBmdW5jdGlvbiAoanNvbk9iamVjdCkgewogICAgICAgICByZXR1cm4g
anNvbk9iamVjdC5wYXJzZSgnWzEsMl0nKTsKICAgICB9CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0
cy9mYXN0L2pzL3Jlc291cmNlcy9KU09OLXBhcnNlLmpzIGIvTGF5b3V0VGVzdHMvZmFzdC9qcy9y
ZXNvdXJjZXMvSlNPTi1wYXJzZS5qcwppbmRleCA4Yzc0MTg3Li5lODNjYjcwIDEwMDY0NAotLS0g
YS9MYXlvdXRUZXN0cy9mYXN0L2pzL3Jlc291cmNlcy9KU09OLXBhcnNlLmpzCisrKyBiL0xheW91
dFRlc3RzL2Zhc3QvanMvcmVzb3VyY2VzL0pTT04tcGFyc2UuanMKQEAgLTg5LDYgKzg5LDcgQEAg
ZnVuY3Rpb24gY3JlYXRlVGVzdHMoKSB7CiAgICAgcmVzdWx0LnB1c2goZnVuY3Rpb24oanNvbk9i
amVjdCl7CiAgICAgICAgIHJldHVybiBqc29uT2JqZWN0LnBhcnNlKCdbMSxdJyk7CiAgICAgfSk7
CisgICAgcmVzdWx0W3Jlc3VsdC5sZW5ndGggLSAxXS50aHJvd3MgPSB0cnVlOwogICAgIHJlc3Vs
dC5wdXNoKGZ1bmN0aW9uKGpzb25PYmplY3QpewogICAgICAgICByZXR1cm4ganNvbk9iamVjdC5w
YXJzZSgnWzEsMl0nKTsKICAgICB9KTs=
</data>
<flag name="review"
          id="19678"
          type_id="1"
          status="+"
          setter="ap"
    />
          </attachment>
      

    </bug>

</bugzilla>