fixes to VB and NC:

- better logging
- better DB transation code (Entities instead of EntityWrapper)
- separated host and instance polling
- coverity fixes

storage/blobstore.c

@@ -4074,7 +4074,7 @@ static int open_blobstore (const char * path, blobstore ** bs, const char * name
         blobstore_set_error_function ( &bs_errors ); 
 
         * bs = blobstore_open (path, 0, BLOBSTORE_FLAG_RDWR, BLOBSTORE_FORMAT_ANY, BLOBSTORE_REVOCATION_ANY, BLOBSTORE_SNAPSHOT_ANY);
-        if (bs==NULL) {
+        if (* bs == NULL) {
             fprintf (stderr, "failed to open %s blobstore in '%s': %s\n", name, path, blobstore_get_error_str(blobstore_get_error()));
             exit (1);
         }

storage/vbr.c

@@ -194,7 +194,8 @@ parse_rec ( // parses the VBR as supplied by a client or user, checks values, an
 
     // identify the type of resource location from location string
     int error = OK;
-    if (strcasestr (vbr->resourceLocation, "http://") == vbr->resourceLocation) { 
+    if (strcasestr (vbr->resourceLocation, "http://") == vbr->resourceLocation ||
+        strcasestr (vbr->resourceLocation, "https://") == vbr->resourceLocation) { 
         if (strcasestr (vbr->resourceLocation, "/services/Walrus/")) {
             vbr->locationType = NC_LOCATION_WALRUS;
         } else {

storage/walrus.c

@@ -132,12 +132,13 @@ static int walrus_request_timeout (const char * walrus_op, const char * verb, co
 
     /* isolate the PATH in the URL as it will be needed for signing */
     char * url_path;
-    if (strncasecmp (url, "http://", 7)!=0) {
-        logprintfl (EUCAERROR, "{%u} walrus_request: URL must start with http://...\n",(unsigned int)pthread_self());
+    if (strncasecmp (url, "http://", 7)!=0 && 
+        strncasecmp (url, "https://", 8)!=0) {
+        logprintfl (EUCAERROR, "{%u} walrus_request: URL must start with http(s)://...\n",(unsigned int)pthread_self());
         pthread_mutex_unlock(&wreq_mutex);
         return code;
     }
-    if ((url_path=strchr(url+7, '/'))==NULL) { /* find first '/' after hostname */
+    if ((url_path=strchr(url+8, '/'))==NULL) { /* find first '/' after hostname */
         logprintfl (EUCAERROR, "{%u} walrus_request: URL has no path\n",(unsigned int)pthread_self());
         pthread_mutex_unlock(&wreq_mutex);
         return code;

util/misc.c

@@ -2061,7 +2061,7 @@ int get_blkid (const char * dev_path, char * uuid, unsigned int uuid_size)
 char parse_boolean (const char * s)
 {
     char * lc = strduplc (s);
-    char val;
+    char val = 0;
 
     if (strcmp (lc, "y")==0 ||
         strcmp (lc, "yes")==0 ||