Properly copy fields array before sorting in toObject, fixes #729

protobufjs · Mar 27, 2017 · f755d36 · f755d36
1 parent 8de21e1
commit f755d36
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/converter.js b/src/converter.js
@@ -193,7 +193,7 @@ function genValuePartial_toObject(gen, field, fieldIndex, prop) {
  */
 converter.toObject = function toObject(mtype) {
     /* eslint-disable no-unexpected-multiline, block-scoped-var, no-redeclare */
-    var fields = mtype.fieldsArray.sort(util.compareFieldsById);
+    var fields = mtype.fieldsArray.slice().sort(util.compareFieldsById);
     if (!fields.length)
         return util.codegen()("return {}");
     var gen = util.codegen("m", "o")