[Bug 1279388] Re: [MIR] thermald

2014-03-17 Thread Adam Conrad
MIR approved, seeded to supported-kernel-common and promoted to main: Override component to main thermald 1.1~rc2-10 in trusty: universe/misc - main thermald 1.1~rc2-10 in trusty amd64: universe/admin/optional/100% - main thermald 1.1~rc2-10 in trusty i386: universe/admin/optional/100% - main

[Bug 1279388] Re: [MIR] thermald

2014-03-14 Thread Colin King
And docs are now: https://wiki.ubuntu.com/Kernel/PowerManagement/ThermalIssues -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1279388 Title: [MIR] thermald To manage notifications about this bug go

[Bug 1279388] Re: [MIR] thermald

2014-03-13 Thread Colin King
thermald rc2-10 now contains these outstanding fixes (execept for the Android specific issues that don't apply to the build in Ubuntu): https://launchpadlibrarian.net/169341284/thermald_1.1~rc2-8_1.1~rc2-10.diff.gz -- You received this bug notification because you are a member of Ubuntu Bugs,

[Bug 1279388] Re: [MIR] thermald

2014-03-10 Thread Seth Arnold
The unsafe pid file handling has been assigned CVE-2014-2312:http://www.openwall.com/lists/oss-security/2014/03/09/2 ** CVE added: http://www.cve.mitre.org/cgi- bin/cvename.cgi?name=2014-2312 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to

[Bug 1279388] Re: [MIR] thermald

2014-03-10 Thread Seth Arnold
Colin, I'm content with the daemon(3) function; I had overlooked it because I was looking for the 'usual sequence' of calls as laid out in Advanced Programming in the Unix Environment: - umask() - chdir() - closing unneeded filedescriptors - ensuring 0, 1, 2 are open to something that makes

[Bug 1279388] Re: [MIR] thermald

2014-03-10 Thread Colin King
Thanks Seth. I've fixed up the outstanding issues and send patches upstream as well as a few more minor memory leaks I found using some different static code analysers. I'll try and get these fixes into the next release of thermald and uploaded in a few more days once upstream has sanity checked

[Bug 1279388] Re: [MIR] thermald

2014-03-10 Thread Seth Arnold
Colin, very cool! Thanks. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1279388 Title: [MIR] thermald To manage notifications about this bug go to:

[Bug 1279388] Re: [MIR] thermald

2014-03-08 Thread Colin King
Thanks Seth for that very instructive list of issues. I will work on these as soon as I can next week. BTW, what is the suggested correct way to daemonize? Any examples I can look at? Thanks again! -- You received this bug notification because you are a member of Ubuntu Bugs, which is

[Bug 1279388] Re: [MIR] thermald

2014-03-07 Thread Seth Arnold
I reviewed thermald version 1.1~rc2-8 as checked into Ubuntu Trusty. This should not be considered a full security audit, but rather a quick gauge of maintainability. - thermald provides a daemon to monitor machine temperature sensors and modify different sysfs-exposed properties to control

[Bug 1279388] Re: [MIR] thermald

2014-03-07 Thread Seth Arnold
I'm sorry, I made a mistake and overlooked the daemon(0,0) call in the startup sequence; this does daemonize properly. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1279388 Title: [MIR] thermald

[Bug 1279388] Re: [MIR] thermald

2014-02-27 Thread Colin King
Seth, any progress on the review? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1279388 Title: [MIR] thermald To manage notifications about this bug go to:

[Bug 1279388] Re: [MIR] thermald

2014-02-27 Thread Seth Arnold
No progress to report yet, sorry. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1279388 Title: [MIR] thermald To manage notifications about this bug go to:

[Bug 1279388] Re: [MIR] thermald

2014-02-12 Thread Adam Conrad
Given that this is a root daemon, I'd like a security audit of this before I go ahead and finish off the MIR for mundane packaging issues. Reassigning to the security team. ** Changed in: thermald (Ubuntu) Assignee: MIR approval team (ubuntu-mir) = Ubuntu Security Team (ubuntu-security) --

[Bug 1279388] Re: [MIR] thermald

2014-02-12 Thread Jamie Strandboge
** Changed in: thermald (Ubuntu) Assignee: Ubuntu Security Team (ubuntu-security) = Seth Arnold (seth-arnold) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1279388 Title: [MIR] thermald To