This is an archive of the discontinued LLVM Phabricator instance.

s390 CVE-2016-2143 whitelist for RHEL kernels
ClosedPublic

Authored by jakubjelinek on Feb 10 2017, 5:31 AM.

Diff Detail

Event Timeline

jakubjelinek created this revision.Feb 10 2017, 5:31 AM
kcc edited edge metadata.Feb 10 2017, 2:29 PM
  • 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.
In D29825#674022, @kcc wrote:
  • please upload patches with full context, otherwise they are harder to review

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.

kcc added a comment.Feb 10 2017, 2:46 PM

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.

kcc accepted this revision.Feb 10 2017, 3:00 PM

LGTM

This revision is now accepted and ready to land.Feb 10 2017, 3:00 PM
kcc closed this revision.Feb 10 2017, 3:02 PM