From e1966ce1820dfacb11d8cd4ea7f04125a6694c1a Mon Sep 17 00:00:00 2001
From: JumpMaster <kevin@cooper.uk.com>
Date: Sun, 21 Sep 2014 10:19:15 +0100
Subject: [PATCH] Version 1.4

---
 src/js/pebble-js-app.js | 11 ++++++++---
 src/main.c              | 11 +++++++++--
 2 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/src/js/pebble-js-app.js b/src/js/pebble-js-app.js
index 2827465..f126357 100644
--- a/src/js/pebble-js-app.js
+++ b/src/js/pebble-js-app.js
@@ -17,7 +17,7 @@ function loadLocalVariables() {
 	for (var i=0; i<MAX_OTP; i++)
 	{
 		var tempKey = localStorage.getItem("secret_pair"+i);
-		if (tempKey)
+		if (tempKey !== null)
 			otp_count++;
 		else
 			break;
@@ -137,6 +137,9 @@ Pebble.addEventListener("appmessage",
 							});
 
 Pebble.addEventListener('showConfiguration', function(e) {
+	// Disable timeout while in configuration menu
+	sendAppMessage({"idle_timeout":0});
+	
 	var url = 'http://oncloudvirtual.com/pebble/pebbleauth/v'+
 		app_version+'/'+
 		'?otp_count='+otp_count+
@@ -158,7 +161,7 @@ Pebble.addEventListener("webviewclosed",
 								if(!isNaN(configuration.delete_all)) {
 									if (debug)
 										console.log("INFO: Delete all requested");
-									for (i = 0; i <= MAX_OTP;i++) {
+									for (i = 0; i < MAX_OTP;i++) {
 										if (debug)
 											console.log("INFO: Deleting key "+i);
 										localStorage.removeItem('secret_pair'+i);
@@ -192,8 +195,10 @@ Pebble.addEventListener("webviewclosed",
 									
 									idle_timeout = configuration.idle_timeout;
 									localStorage.setItem("idle_timeout",idle_timeout);
-									config.idle_timeout = idle_timeout;
 								}
+								// Always send idle_timeout to reset it after
+								// it was disabled while configuring
+								config.idle_timeout = idle_timeout;
 								
 								if(configuration.label && configuration.secret) {
 									var secret = configuration.secret
diff --git a/src/main.c b/src/main.c
index e834729..c07d99c 100644
--- a/src/main.c
+++ b/src/main.c
@@ -117,6 +117,12 @@ void expand_key(char *inputString, bool new_code) {
 	}
 	otp_key[outputChar] = '\0';
 	
+	// If the label or key are null ignore them
+	if (otp_label == '\0' || otp_key == '\0') {
+		APP_LOG(APP_LOG_LEVEL_DEBUG, "INFO: NULL key or label, ignoring");
+		return;
+	}
+	
 	bool updating_label = false;
 	if (new_code) {
 		for(unsigned int i = 0; i < watch_otp_count; i++) {
@@ -664,7 +670,7 @@ static void in_received_handler(DictionaryIterator *iter, void *context) {
 		if (DEBUG)
 			APP_LOG(APP_LOG_LEVEL_DEBUG, "INFO: Delete all requested");
 		
-		for (unsigned int i = 0; i <= MAX_OTP; i++) {
+		for (unsigned int i = 0; i < MAX_OTP; i++) {
 			if (DEBUG)
 				APP_LOG(APP_LOG_LEVEL_DEBUG, "INFO: Deleting key: %d", i);
 			persist_delete(PS_SECRET+i);
@@ -776,7 +782,8 @@ static void in_received_handler(DictionaryIterator *iter, void *context) {
 		if (idle_timeout != idle_timeout_value) {
 			idle_timeout = idle_timeout_value;
 			persist_write_int(PS_IDLE_TIMEOUT, idle_timeout);
-
+			resetIdleTime();
+			
 			if (DEBUG)
 				APP_LOG(APP_LOG_LEVEL_DEBUG, "INFO: Idle Timeout: %d", idle_timeout);
 		}