aboutsummaryrefslogtreecommitdiffstats
path: root/lib/snmp/src/agent/snmpa_acm.erl
diff options
context:
space:
mode:
authorIngela Anderton Andin <[email protected]>2016-01-12 10:20:41 +0100
committerIngela Anderton Andin <[email protected]>2016-01-21 11:41:42 +0100
commit290dffee0f7af744c86d69234fb2d8ed1c4b8a6b (patch)
treec99f5a31bf75913dc5639c84c3417af93732f385 /lib/snmp/src/agent/snmpa_acm.erl
parent931b51693132839c3bc0edc3ed01364135293c1b (diff)
downloadotp-290dffee0f7af744c86d69234fb2d8ed1c4b8a6b.tar.gz
otp-290dffee0f7af744c86d69234fb2d8ed1c4b8a6b.tar.bz2
otp-290dffee0f7af744c86d69234fb2d8ed1c4b8a6b.zip
Snmp agent vacmViewTreeFamily with non 'null' mask bugfix
Adresses problems desciribed in http://erlang.org/pipermail/erlang-bugs/2015-April/004891.html http://erlang.org/pipermail/erlang-patches/2015-April/004766.html The original patch also suggested the following fix. However we need more information to be able to understand if this is correct or not, and it does not have priority at the moment for us to investigate. @@ -816,7 +816,7 @@ next_node(D, {tree, Tree, {table_entry, _MibName}}, "~n RevOidSoFar: ~p" "~n MibView: ~p", [size(Tree), Oid, RevOidSoFar, MibView]), OidSoFar = lists:reverse(RevOidSoFar), - case snmpa_acm:is_definitely_not_in_mib_view(OidSoFar, MibView) of + case snmpa_acm:is_definitely_not_in_mib_view(OidSoFar ++ Oid, MibView) of true -> ?vdebug("next_node(tree,table_entry) -> not in mib view",[]), false;
Diffstat (limited to 'lib/snmp/src/agent/snmpa_acm.erl')
-rw-r--r--lib/snmp/src/agent/snmpa_acm.erl4
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/snmp/src/agent/snmpa_acm.erl b/lib/snmp/src/agent/snmpa_acm.erl
index 7327575846..0264c6a992 100644
--- a/lib/snmp/src/agent/snmpa_acm.erl
+++ b/lib/snmp/src/agent/snmpa_acm.erl
@@ -280,7 +280,7 @@ validate_mib_view(Oid, MibView) ->
end.
get_largest_family([{SubTree, Mask, Type} | T], Oid, Res) ->
- case check_mask(Oid, SubTree, Mask) of
+ case check_mask(Oid, SubTree, snmp_view_based_acm_mib:emask2imask(Mask)) of
true -> get_largest_family(T, Oid, add_res(length(SubTree), SubTree,
Type, Res));
false -> get_largest_family(T, Oid, Res)
@@ -345,7 +345,7 @@ validate_all_mib_view([], _MibView) ->
%% intelligent.
%%-----------------------------------------------------------------
is_definitely_not_in_mib_view(Oid, [{SubTree, Mask,?view_included}|T]) ->
- case check_maybe_mask(Oid, SubTree, Mask) of
+ case check_maybe_mask(Oid, SubTree, snmp_view_based_acm_mib:emask2imask(Mask)) of
true -> false;
false -> is_definitely_not_in_mib_view(Oid, T)
end;