summaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab+huawei@kernel.org>2021-09-27 16:49:49 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2021-09-28 13:45:36 +0300
commit3a1cc06c0e07065b06ba1af28eb48886e099e195 (patch)
tree36574fe35b5c39edaeab320ce5cbe9c11c27b7db /scripts
parent3cb1feadbffd536b58373312d93feae08c010193 (diff)
downloadlinux-3a1cc06c0e07065b06ba1af28eb48886e099e195.tar.xz
scripts: get_abi.pl: produce an error if the ref tree is broken
The logic under graph_add_file should create, for every entry, a __name name array for all entries of the tree. If this fails, the symlink parsing will break. Add an error if this ever happens. While here, improve the output of data dumper to be more compact and to avoid displaying things like $VAR1=. Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org> Link: https://lore.kernel.org/r/e7dd4d70e206723455d50c851802c8bb6c34941d.1632750315.git.mchehab+huawei@kernel.org Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'scripts')
-rwxr-xr-xscripts/get_abi.pl10
1 files changed, 9 insertions, 1 deletions
diff --git a/scripts/get_abi.pl b/scripts/get_abi.pl
index c191c024f052..26a3f8ff566a 100755
--- a/scripts/get_abi.pl
+++ b/scripts/get_abi.pl
@@ -27,6 +27,9 @@ my $dbg_what_open = 2;
my $dbg_dump_abi_structs = 4;
my $dbg_undefined = 8;
+$Data::Dumper::Indent = 1;
+$Data::Dumper::Terse = 1;
+
#
# If true, assumes that the description is formatted with ReST
#
@@ -597,7 +600,6 @@ sub graph_add_link {
my @queue;
my %seen;
- my $base_name;
my $st;
push @queue, $file_ref;
@@ -611,6 +613,12 @@ sub graph_add_link {
next if $seen{$$v{$c}};
next if ($c eq "__name");
+ if (!defined($$v{$c}{"__name"})) {
+ printf STDERR "Error: Couldn't find a non-empty name on a children of $file/.*: ";
+ print STDERR Dumper(%{$v});
+ exit;
+ }
+
# Add new name
my $name = @{$$v{$c}{"__name"}}[0];
if ($name =~ s#^$file/#$link/#) {