Closed Bug 1277427 Opened 4 years ago Closed 4 years ago

DOM: Fix constants missing for splice

Categories

(Core :: DOM: Core & HTML, defect)

All
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed

People

(Reporter: hev, Assigned: hev)

References

Details

Attachments

(1 file)

*************************
A coding exception was thrown and uncaught in a Task.

Full message: TypeError: Argument 5 of splice is undefined
Full stack: ffi@resource://gre/modules/osfile/osfile_shared_allthreads.jsm:1109:17
pump_splice@resource://gre/modules/osfile/osfile_unix_front.jsm:603:18
copy@resource://gre/modules/osfile/osfile_unix_front.jsm:658:23
Agent.write@resource:///modules/sessionstore/SessionWorker.js:215:9
worker.dispatch@resource:///modules/sessionstore/SessionWorker.js:21:24
anonymous/AbstractWorker.prototype.handleMessage@resource://gre/modules/workers/PromiseWorker.js:122:16
@resource:///modules/sessionstore/SessionWorker.js:30:41
EventListener.handleEvent*@resource:///modules/sessionstore/SessionWorker.js:30:1

*************************
console.error: 
  Could not write session state file 
  Message: TypeError: Argument 5 of splice is undefined
  Stack:
    ffi@resource://gre/modules/osfile/osfile_shared_allthreads.jsm:1109:17
pump_splice@resource://gre/modules/osfile/osfile_unix_front.jsm:603:18
copy@resource://gre/modules/osfile/osfile_unix_front.jsm:658:23
Agent.write@resource:///modules/sessionstore/SessionWorker.js:215:9
worker.dispatch@resource:///modules/sessionstore/SessionWorker.js:21:24
anonymous/AbstractWorker.prototype.handleMessage@resource://gre/modules/workers/PromiseWorker.js:122:16
@resource:///modules/sessionstore/SessionWorker.js:30:41
EventListener.handleEvent*@resource:///modules/sessionstore/SessionWorker.js:30:1

  ffi@resource://gre/modules/osfile/osfile_shared_allthreads.jsm:1109:17
pump_splice@resource://gre/modules/osfile/osfile_unix_front.jsm:603:18
copy@resource://gre/modules/osfile/osfile_unix_front.jsm:658:23
Agent.write@resource:///modules/sessionstore/SessionWorker.js:215:9
worker.dispatch@resource:///modules/sessionstore/SessionWorker.js:21:24
anonymous/AbstractWorker.prototype.handleMessage@resource://gre/modules/workers/PromiseWorker.js:122:16
@resource:///modules/sessionstore/SessionWorker.js:30:41
EventListener.handleEvent*@resource:///modules/sessionstore/SessionWorker.js:30:1

Looks pump_splice access a undefined constant variable SPLICE_F_MORE:
https://dxr.mozilla.org/mozilla-central/rev/4d63dde701b47b8661ab7990f197b6b60e543839/toolkit/components/osfile/modules/osfile_unix_front.jsm#605
$ cat /tmp/t.js
var Cu = Components.utils;
var SharedAll = Cu.import("resource://gre/modules/osfile/osfile_shared_allthreads.jsm");
var Const = SharedAll.Constants.libc;
print(Const.SPLICE_F_MORE);

$ ./run-mozilla.sh ./xpcshell -g ../firefox -f /tmp/t.js 
undefined

Patched:
$ ./run-mozilla.sh ./xpcshell -g ../firefox -f /tmp/t.js 
4
Attachment #8758987 - Flags: review?(dteller)
Duplicate of this bug: 1277434
Comment on attachment 8758987 [details] [diff] [review]
0001-DOM-Fix-constants-missing-for-splice.patch

Review of attachment 8758987 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me.
Attachment #8758987 - Flags: review?(dteller) → review+
Pushed by r@hev.cc:
https://hg.mozilla.org/integration/mozilla-inbound/rev/cd31d578be58
DOM: Fix constants missing for splice. r=Yoric
Backout by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4a84c4e71f09
Backed out changeset cd31d578be58 for bustage on a CLOSED TREE
Pushed by r@hev.cc:
https://hg.mozilla.org/integration/mozilla-inbound/rev/212dd0c9cdbb
DOM: Fix consts missing for splice. r=Yoric
https://hg.mozilla.org/mozilla-central/rev/212dd0c9cdbb
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.