This patch adds whitelist for RHEL6 and RHEL7 kernels that are known to have the CVE fixed.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
- please upload patches with full context, otherwise they are harder to review
- the patch does not seem to be properly indented
- is it testable?
- if you expect more patches to follow I encourage you to request LLVM commit access.
Comment Actions
Not sure what you mean by that. This was created with svn diff.
- the patch does not seem to be properly indented
The patch uses tab characters which phabricators screws up. Are you avoiding tabs in the source?
- is it testable?
Only by trying to run on the specific RHEL kernels. It is as testable as all the other cases in the list, I bet you have only one kernel installed on the bot, there are many cases in the function already.
- if you expect more patches to follow I encourage you to request LLVM commit access.
This is the last patch for now.
Comment Actions
Not sure what you mean by that. This was created with svn diff.
svn diff gives very little context. I guess you can do "svn diff --diff-cmd=diff -x -U1000" os something like this.
I use 'arc diff' which does all that for me. http://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-command-line
Are you avoiding tabs in the source?
Yes.