upstart-local-bridge not handling all events sent to it
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
upstart |
Fix Committed
|
Undecided
|
James Hunt | ||
upstart (Ubuntu) |
Fix Released
|
Undecided
|
James Hunt |
Bug Description
I have this job: http://
I'm running this script: http://
And this is what I get: http://
From it, the property says it's running, the android side (upstart-
ProblemType: Bug
DistroRelease: Ubuntu 13.10
Package: upstart 1.10-0ubuntu1
Uname: Linux 3.4.0-4-manta armv7l
ApportVersion: 2.12.5-0ubuntu1
Architecture: armhf
Date: Thu Oct 3 18:53:23 2013
InstallationDate: Installed on 2013-10-03 (0 days ago)
InstallationMedia: Ubuntu Saucy Salamander (development branch) - armhf (20131003.1)
MarkForUpload: True
ProcEnviron:
TERM=linux
PATH=(custom, no user)
ProcKernelCmdline: vmalloc=512M debug_core.
SourcePackage: upstart
UpgradeStatus: No upgrade log present (probably fresh install)
UpstartBugCategory: System
modified.
mtime.conffile.
upstart.
upstart.
upstart.
upstart.
upstart.
upstart.
Related branches
Changed in upstart (Ubuntu): | |
assignee: | nobody → James Hunt (jamesodhunt) |
Changed in upstart: | |
assignee: | nobody → James Hunt (jamesodhunt) |
status: | New → Fix Committed |
One thing I did noticed when looking at the code is that the 'socket_reader' function seems to expect the input buffer to be one-line only, and not a block (buf could be a block when reading data from the socket).
As a result upstart_emit_event can emit 'buf' containing more than one event at the same time (including "\n"), which might cause issues later on.
As I'm not that familiar with the code, will let James to comment on the subject.