From 0dd9c193ecb0b30fbbb31e70c8b2087e937d688d Mon Sep 17 00:00:00 2001 From: soh_cah_toa Date: Wed, 23 Apr 2014 12:53:41 -0400 Subject: [PATCH] Appended ! to #remove_rule and #remove_ruleset. Adhering to the Ruby convention, this indicates that these methods mutate the receiver and, therefore, should be considered dangerous. --- extensions/dns/dns.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/extensions/dns/dns.rb b/extensions/dns/dns.rb index 30940eff9..c280adf0d 100644 --- a/extensions/dns/dns.rb +++ b/extensions/dns/dns.rb @@ -72,7 +72,7 @@ module BeEF # @param id [String] rule identifier # # @return [Boolean] true if rule was removed, otherwise false - def remove_rule(id) + def remove_rule!(id) @lock.synchronize do rule = BeEF::Core::Models::Dns::Rule.get(id) rule.nil? ? false : rule.destroy @@ -96,7 +96,7 @@ module BeEF # Removes the entire DNS ruleset. # # @return [Boolean] true if ruleset was destroyed, otherwise false - def remove_ruleset + def remove_ruleset! @lock.synchronize { BeEF::Core::Models::Dns::Rule.destroy } end