From b5c4f461bc8a2ad86cb55ce9a01b5baf60c200b7 Mon Sep 17 00:00:00 2001
From: Robert Osfield <robert@openscenegraph.com>
Date: Wed, 4 Jan 2017 12:55:15 +0000
Subject: [PATCH] Fixed unitialized variable access to premature calling of
 computeMatrix().

---
 src/osgSim/Sector.cpp | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/osgSim/Sector.cpp b/src/osgSim/Sector.cpp
index 7ddc7cef3..029e2ce9b 100644
--- a/src/osgSim/Sector.cpp
+++ b/src/osgSim/Sector.cpp
@@ -218,14 +218,17 @@ float ConeSector::operator() (const osg::Vec3& eyeLocal) const
 //
 // DirectionalSector
 //
-DirectionalSector::DirectionalSector(const osg::Vec3& direction,float horizLobeAngle, float vertLobeAngle, float lobeRollAngle, float fadeAngle):
+DirectionalSector::DirectionalSector(const osg::Vec3& direction, float horizLobeAngle, float vertLobeAngle, float lobeRollAngle, float fadeAngle):
             Sector()
 {
-    setDirection(direction);
-    setHorizLobeAngle(horizLobeAngle);
-    setVertLobeAngle(vertLobeAngle);
-    setLobeRollAngle(lobeRollAngle);
+    _direction = direction;
+    _cosHorizAngle = cos(horizLobeAngle*0.5);
+    _cosVertAngle = cos(vertLobeAngle*0.5);
+    _rollAngle = lobeRollAngle;
+
     setFadeAngle(fadeAngle);
+
+    computeMatrix();
 }
 
 void DirectionalSector::computeMatrix()