Re: Review Request 125717: initialize socket size with correct value
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125717/ --- (Updated Oct. 21, 2015, 5:38 a.m.) Status -- This change has been marked as submitted. Review request for KDE Frameworks and Martin Klapetek. Changes --- Submitted with commit f1e6b9d168281196010c44af2eae4587c1d2d088 by Harald Sitter to branch master. Bugs: 351805 https://bugs.kde.org/show_bug.cgi?id=351805 Repository: kwallet Description --- to quote man 2 accept: > The addrlen argument is a value-result argument: the caller must > initialize it to contain the size (in bytes) of the structure pointed > to by addr; on return it will contain the actual size of the peer > address. If addrlen is not correct we may get EINVAL on trying to connect to the environment socket which in turn results in a broken environment of the daemon when started through pam as it will inherit the DM environment. This doesn't have to happen, it does however reproducibly with nvidia-352 on Kubuntu 15.10. Why or how nvidia plays into this is not entirely clear, best bet is that it simply is a coincidence where nvidia would have something allocated in the memory beforehand and since the addrlen stack variable was not explicitly initialized to anything it would could potentially be negative (EINVAL condition) or wrong such that it meets an internal error check within accept(). BUG: 351805 Diffs - src/runtime/kwalletd/main.cpp 39b5ae97ae0984a9acb1efffa73e492f4b3e6a71 Diff: https://git.reviewboard.kde.org/r/125717/diff/ Testing --- - got error with Kubuntu build of kwallet using nvidia driver - applied patch - no longer got error in multiple test runs - removed patch and rebuilt kwallet - error returns Thanks, Harald Sitter ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 125717: initialize socket size with correct value
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125717/#review87133 --- Ship it! Ship It! - Martin Klapetek On Oct. 20, 2015, 7:23 a.m., Harald Sitter wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125717/ > --- > > (Updated Oct. 20, 2015, 7:23 a.m.) > > > Review request for KDE Frameworks and Martin Klapetek. > > > Bugs: 351805 > https://bugs.kde.org/show_bug.cgi?id=351805 > > > Repository: kwallet > > > Description > --- > > to quote man 2 accept: > > The addrlen argument is a value-result argument: the caller must > > initialize it to contain the size (in bytes) of the structure pointed > > to by addr; on return it will contain the actual size of the peer > > address. > > If addrlen is not correct we may get EINVAL on trying to connect to the > environment socket which in turn results in a broken environment of the > daemon when started through pam as it will inherit the DM environment. > This doesn't have to happen, it does however reproducibly with nvidia-352 > on Kubuntu 15.10. Why or how nvidia plays into this is not entirely clear, > best bet is that it simply is a coincidence where nvidia would have > something allocated in the memory beforehand and since the addrlen > stack variable was not explicitly initialized to anything it would could > potentially be negative (EINVAL condition) or wrong such that it meets > an internal error check within accept(). > > BUG: 351805 > > > Diffs > - > > src/runtime/kwalletd/main.cpp 39b5ae97ae0984a9acb1efffa73e492f4b3e6a71 > > Diff: https://git.reviewboard.kde.org/r/125717/diff/ > > > Testing > --- > > - got error with Kubuntu build of kwallet using nvidia driver > - applied patch > - no longer got error in multiple test runs > - removed patch and rebuilt kwallet > - error returns > > > Thanks, > > Harald Sitter > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Review Request 125717: initialize socket size with correct value
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125717/ --- Review request for KDE Frameworks and Martin Klapetek. Bugs: 351805 https://bugs.kde.org/show_bug.cgi?id=351805 Repository: kwallet Description --- to quote man 2 accept: > The addrlen argument is a value-result argument: the caller must > initialize it to contain the size (in bytes) of the structure pointed > to by addr; on return it will contain the actual size of the peer > address. If addrlen is not correct we may get EINVAL on trying to connect to the environment socket which in turn results in a broken environment of the daemon when started through pam as it will inherit the DM environment. This doesn't have to happen, it does however reproducibly with nvidia-352 on Kubuntu 15.10. Why or how nvidia plays into this is not entirely clear, best bet is that it simply is a coincidence where nvidia would have something allocated in the memory beforehand and since the addrlen stack variable was not explicitly initialized to anything it would could potentially be negative (EINVAL condition) or wrong such that it meets an internal error check within accept(). BUG: 351805 Diffs - src/runtime/kwalletd/main.cpp 39b5ae97ae0984a9acb1efffa73e492f4b3e6a71 Diff: https://git.reviewboard.kde.org/r/125717/diff/ Testing --- - got error with Kubuntu build of kwallet using nvidia driver - applied patch - no longer got error in multiple test runs - removed patch and rebuilt kwallet - error returns Thanks, Harald Sitter ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel