From: "Tobin C. Harding" <me@tobin.cc>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: "Tobin C. Harding" <me@tobin.cc>,
"Jason A. Donenfeld" <Jason@zx2c4.com>,
Theodore Ts'o <tytso@mit.edu>, Kees Cook <keescook@chromium.org>,
Paolo Bonzini <pbonzini@redhat.com>,
Tycho Andersen <tycho@docker.com>,
"Roberts, William C" <william.c.roberts@intel.com>,
Tejun Heo <tj@kernel.org>,
Jordan Glover <Golden_Miller83@protonmail.ch>,
Greg KH <gregkh@linuxfoundation.org>,
Petr Mladek <pmladek@suse.com>, Joe Perches <joe@perches.com>,
Ian Campbell <ijc@hellion.org.uk>,
Sergey Senozhatsky <sergey.senozhatsky@gmail.com>,
Catalin Marinas <catalin.marinas@arm.com>,
Will Deacon <wilal.deacon@arm.com>,
Steven Rostedt <rostedt@goodmis.org>,
Chris Fries <cfries@google.com>,
Dave Weinstein <olorin@google.com>,
Daniel Micay <danielmicay@gmail.com>,
Subject: [PATCH 5/7] scripts/leaking_addresses: add emailing results
Date: Wed, 8 Nov 2017 14:37:37 +1100 [thread overview]
Message-ID: <1510112259-11572-6-git-send-email-me@tobin.cc> (raw)
In-Reply-To: <1510112259-11572-1-git-send-email-me@tobin.cc>
Developers may not have the time (or inclination) to investigate script
output. This information is, however, useful. If we add functionality to
the script to email results for further investigation.
Add --send-report flag to email scan results (to Tobin C. Harding).
Signed-off-by: Tobin C. Harding <me@tobin.cc>
---
scripts/leaking_addresses.pl | 42 ++++++++++++++++++++++++++++++++++++++++++
1 file changed, 42 insertions(+)
mode change 100755 => 100644 scripts/leaking_addresses.pl
diff --git a/scripts/leaking_addresses.pl b/scripts/leaking_addresses.pl
old mode 100755
new mode 100644
index 4c31e935319b..e43105662306
--- a/scripts/leaking_addresses.pl
+++ b/scripts/leaking_addresses.pl
@@ -34,6 +34,7 @@ my $output = "scan.out";
my $suppress_dmesg = 0;
my $squash_by_path = 0;
my $raw = 0;
+my $send_report = 0;
my $help = 0;
my $debug = 0;
@@ -90,6 +91,7 @@ Options:
--suppress-dmesg Do not show dmesg results.
--squash-by-path Show one result per unique path.
--raw Show raw results.
+ --send-report Submit raw results for someone else to worry about.
-d, --debug Display debugging output.
-h, --help, --version Display this help and exit.
@@ -103,6 +105,7 @@ GetOptions(
'suppress-dmesg' => \$suppress_dmesg,
'squash-by-path' => \$squash_by_path,
'raw' => \$raw,
+ 'send-report' => \$send_report,
'd|debug' => \$debug,
'h|help' => \$help,
'version' => \$help
@@ -124,6 +127,12 @@ if ($command eq 'scan') {
scan();
}
+if ($send_report) {
+ send_report();
+ print "Raw scan results sent, thank you.\n";
+ exit(0);
+}
+
format_output();
exit 0;
@@ -144,6 +153,39 @@ sub scan
select STDOUT;
}
+sub send_report
+{
+ my $subject = 'LEAK REPORT';
+ my $email = 'leaks@tobin.cc';
+
+ my $message = sprintf("kptr_restrict: %s\n", get_kptr_restrict());
+
+ # Slurp raw results.
+ $message .= do {
+ local $/ = undef;
+ open my $fh, "<", $output
+ or die "could not open $output: $!";
+ <$fh>;
+ };
+
+ open my $mailh, '|-', "mail -s '$subject' $email"
+ or die( "Could not open pipe! $!" );
+
+ print $mailh $message;
+ close $mailh;
+}
+
+sub get_kptr_restrict
+{
+ my $filename = "/proc/sys/kernel/kptr_restrict";
+ my @array = do {
+ open my $fh, "<", $filename
+ or die "could not open $filename: $!";
+ <$fh>;
+ };
+ return $array[0];
+}
+
sub is_false_positive
{
my ($match) = @_;
--
2.7.4
next prev parent reply other threads:[~2017-11-08 3:37 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-08 3:37 [PATCH 0/7] scripts/leaking_addresses: add summary report Tobin C. Harding
2017-11-08 3:37 ` [PATCH 1/7] scripts/leaking_addresses: use tabs not spaces Tobin C. Harding
2017-11-08 3:37 ` [PATCH 2/7] scripts/leaking_addresses: remove dead code Tobin C. Harding
2017-11-08 3:37 ` [PATCH 3/7] scripts/leaking_addresses: remove command line options Tobin C. Harding
2017-11-08 3:37 ` [PATCH 4/7] scripts/leaking_addresses: add reporting Tobin C. Harding
2017-11-08 10:42 ` Petr Mladek
2017-11-09 0:51 ` Tobin C. Harding
2017-11-08 3:37 ` Tobin C. Harding [this message]
2017-11-08 10:16 ` [PATCH 5/7] scripts/leaking_addresses: add emailing results Petr Mladek
2017-11-08 11:51 ` Greg KH
2017-11-09 0:58 ` Tobin C. Harding
2017-11-08 3:37 ` [PATCH 6/7] scripts/leaking_addresses: fix comment typo Tobin C. Harding
2017-11-08 3:37 ` [PATCH 7/7] scripts/leaking_addresses: don't parse usbmon Tobin C. Harding
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1510112259-11572-6-git-send-email-me@tobin.cc \
--to=me@tobin.cc \
--cc=Golden_Miller83@protonmail.ch \
--cc=Jason@zx2c4.com \
--cc=catalin.marinas@arm.com \
--cc=cfries@google.com \
--cc=danielmicay@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=ijc@hellion.org.uk \
--cc=joe@perches.com \
--cc=keescook@chromium.org \
--cc=olorin@google.com \
--cc=pbonzini@redhat.com \
--cc=pmladek@suse.com \
--cc=rostedt@goodmis.org \
--cc=sergey.senozhatsky@gmail.com \
--cc=tj@kernel.org \
--cc=torvalds@linux-foundation.org \
--cc=tycho@docker.com \
--cc=tytso@mit.edu \
--cc=wilal.deacon@arm.com \
--cc=william.c.roberts@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).