{"comment":{"commit_id":"60aff3e236acccd344b8fe1b68079290539bae92","position":null,"created_at":"2011-12-05T23:13:01Z","updated_at":"2011-12-05T23:13:01Z","user":{"gravatar_id":"80ae8f3b80319adaf9d5a834ec574058","id":616784,"url":"https://api.github.com/users/rickyz","login":"rickyz","avatar_url":"https://secure.gravatar.com/avatar/80ae8f3b80319adaf9d5a834ec574058?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"body":"Hey, sorry - I missed this comment.  Good catch on the other unpack call - I completed missed that from the patch.  From looking at the SELinux code for reading modules, everything should be little endian, so that L needs to be changed to a V as well.\r\n\r\nI should have mentioned that I haven't had a chance to test the patch, as I didn't have a non-production PPC machine.  From the looks of it, the original patch would have errored out slightly later without fixing selmod_readnext as well.","id":770717,"path":null,"line":null,"html_url":"https://github.com/rickyz/puppet/commit/60aff3e236#commitcomment-770717","url":"https://api.github.com/repos/rickyz/puppet/comments/770717"}}