Version 3.1.0

When an SGPath is empty, don't bother stat()-ing. This avoids
an uninitialized-memory read inside the C-runtime on Windows.

simgear/io/SVNDirectory.cxx

@@ -319,7 +319,6 @@ void SVNDirectory::mergeUpdateReportDetails(unsigned int depth,
     
     Dir d(localPath);
     if (depth >= MAX_UPDATE_REPORT_DEPTH) {
-        SG_LOG(SG_IO, SG_INFO, localPath << "exceeded MAX_UPDATE_REPORT_DEPTH, cleaning");
         d.removeChildren();
         return;
     }

simgear/misc/sg_path.cxx

@@ -312,13 +312,20 @@ void SGPath::validate() const
     return;
   }
 
+  if (path.empty()) {
+	  _exists = false;
+	  return;
+  }
+
 #ifdef _WIN32
   struct _stat buf ;
-
   bool remove_trailing = false;
-  if ( path.length() > 1 && path[path.length()-1] == '/' )
-      remove_trailing=true;
-  if (_stat (path.substr(0,remove_trailing?path.length()-1:path.length()).c_str(), &buf ) < 0) {
+  string statPath(path);
+  if ((path.length() > 1) && (path.back() == '/')) {
+	  statPath.pop_back();
+  }
+      
+  if (_stat(statPath.c_str(), &buf ) < 0) {
     _exists = false;
   } else {
     _exists = true;

simgear/scene/material/Technique.cxx

@@ -118,7 +118,7 @@ void Technique::validateInContext(GraphicsContext* gc)
     Status oldVal = contextInfo.valid();
     Status newVal = INVALID;
     expression::FixedLengthBinding<1> binding;
-    binding.getBindings()[_contextIdLocation].val.intVal = contextId;
+    binding.getBindings()[_contextIdLocation] = expression::Value((int) contextId);
     if (_validExpression->getValue(&binding))
         newVal = VALID;
     contextInfo.valid.compareAndSwap(oldVal, newVal);

version

@@ -1 +1 @@
-3.00.0
+3.1.0