From d63c675e858542f72880c8a129df3c02eaab58b3 Mon Sep 17 00:00:00 2001
From: "shingil.kang" <shingil.kang@samsung.com>
Date: Mon, 13 Jan 2014 22:51:38 +0900
Subject: [PATCH] CLI: Fixed the prevent issues Fixed "Dereference null return
 value"

Change-Id: Ie2ebb21b2fb4ac9c20228252dd7009ac1de9f295
Signed-off-by: shingil.kang <shingil.kang@samsung.com>
---
 .../ide/subcommands/CreateWebProjectCommand.java   | 115 ++++++++++++---------
 1 file changed, 65 insertions(+), 50 deletions(-)

diff --git a/org.tizen.ncli.ide/src/org/tizen/ncli/ide/subcommands/CreateWebProjectCommand.java b/org.tizen.ncli.ide/src/org/tizen/ncli/ide/subcommands/CreateWebProjectCommand.java
index 9b80da3..5e9a4d1 100644
--- a/org.tizen.ncli.ide/src/org/tizen/ncli/ide/subcommands/CreateWebProjectCommand.java
+++ b/org.tizen.ncli.ide/src/org/tizen/ncli/ide/subcommands/CreateWebProjectCommand.java
@@ -213,10 +213,13 @@ public class CreateWebProjectCommand extends AbstractSubCommand<CreateProjectCom
         }
 
         // get options
-        Options options = tizenAppTemplate.getOptions();
-        if(options == null){
-            log.error("not get options");
-            return;
+        Options options = null;
+        if (tizenAppTemplate != null){
+            options = tizenAppTemplate.getOptions();
+            if(options == null){
+                log.error("not get options");
+                return;
+            }
         }
 
         // get support libraries
@@ -229,23 +232,26 @@ public class CreateWebProjectCommand extends AbstractSubCommand<CreateProjectCom
         String rootPath = pg.extractBuiltinJarFiles(WebConstant.BUILTIN_TEMPLATE_LIBRARIES_FOLDER);
 
         // get library pool
-        libraryPool = new TizenTemplateLibrariesPool(profileInfo, rootPath);
+        if (profileInfo != null)
+        {
+            libraryPool = new TizenTemplateLibrariesPool(profileInfo, rootPath);
 
-        // get template library groups
-        if (libraryPool != null)
-        { // In case of sample, pool is null
-            List<TizenTemplateLibraryGroup> groupList = libraryPool.getLibraryGroupWithDepends(sl);
+            // get template library groups
+            if (libraryPool != null)
+            { // In case of sample, pool is null
+                List<TizenTemplateLibraryGroup> groupList = libraryPool.getLibraryGroupWithDepends(sl);
 
-            // copy libraries
-            for (TizenTemplateLibraryGroup group : groupList)
-            {
-                if (group.getSelectedElement().getKindsOfLibrary() == TizenTemplateKindsOfLibraries.TIZENWEBUIFW)
+                // copy libraries
+                for (TizenTemplateLibraryGroup group : groupList)
                 {
-                    copyTizenWebUIFWLibrary(group);
-                }
-                else
-                {
-                    copyNormalLibrary(group);
+                    if (group.getSelectedElement().getKindsOfLibrary() == TizenTemplateKindsOfLibraries.TIZENWEBUIFW)
+                    {
+                        copyTizenWebUIFWLibrary(group);
+                    }
+                    else
+                    {
+                        copyNormalLibrary(group);
+                    }
                 }
             }
         }
@@ -297,6 +303,9 @@ public class CreateWebProjectCommand extends AbstractSubCommand<CreateProjectCom
             destinationURI = URI.create(destinationURI + group.getSelectedElementValue());
         }
 
+        if (destinationURI == null)
+            return;
+
         // copy must pre-create directory
         if (source.isDirectory())
         {
@@ -587,21 +596,24 @@ public class CreateWebProjectCommand extends AbstractSubCommand<CreateProjectCom
         Platforms platforms = objFactory.createPlatforms();
 
         // set latest profile name
-        platform.setName(latestProfileInfo.getLatestPlatformName());
-        platforms.getPlatform().add(platform);
-        tProject.setPlatforms(platforms);
-
-        // make tizen project description file
-        File file = new File(data.getDestTizenProjectDescriptionFile());
-
-        // marshall
-        try {
-            JAXBContext jaxbContext = JAXBContext.newInstance(ObjectFactory.class.getPackage().getName());
-            Marshaller marshaller = jaxbContext.createMarshaller();
-            marshaller.setProperty(Marshaller.JAXB_FORMATTED_OUTPUT, Boolean.TRUE);
-            marshaller.marshal(tProject, file);
-        } catch (JAXBException e) {
-            log.error("cannot marshal \n{}", e.getMessage());
+        if (latestProfileInfo != null)
+        {
+            platform.setName(latestProfileInfo.getLatestPlatformName());
+            platforms.getPlatform().add(platform);
+            tProject.setPlatforms(platforms);
+
+            // make tizen project description file
+            File file = new File(data.getDestTizenProjectDescriptionFile());
+
+            // marshall
+            try {
+                JAXBContext jaxbContext = JAXBContext.newInstance(ObjectFactory.class.getPackage().getName());
+                Marshaller marshaller = jaxbContext.createMarshaller();
+                marshaller.setProperty(Marshaller.JAXB_FORMATTED_OUTPUT, Boolean.TRUE);
+                marshaller.marshal(tProject, file);
+            } catch (JAXBException e) {
+                log.error("cannot marshal \n{}", e.getMessage());
+            }
         }
     }
 
@@ -655,25 +667,28 @@ public class CreateWebProjectCommand extends AbstractSubCommand<CreateProjectCom
         // get configuration manager
         ITizenConfigurator configurator = unmarshalWidgetConfiguration(configurationFile);
 
-        // set default configuration
-        configurator.setId(widgetId);
-        configurator.setName(projectName, null);
+        if (configurator != null)
+        {
+            // set default configuration
+            configurator.setId(widgetId);
+            configurator.setName(projectName, null);
 
-        // Setting appid of configuration for new project
-        configurator.setApplication(projectName, null, 0);
+            // Setting appid of configuration for new project
+            configurator.setApplication(projectName, null, 0);
 
-        // save
-        try
-        {
-            configurator.doSave();
-        } catch (IOException e)
-        {
-            // TODO Auto-generated catch block
-            log.error(e.getMessage());
-        } catch (ConfigurationException e)
-        {
-            // TODO Auto-generated catch block
-            log.error(e.getMessage());
+            // save
+            try
+            {
+                configurator.doSave();
+            } catch (IOException e)
+            {
+                // TODO Auto-generated catch block
+                log.error(e.getMessage());
+            } catch (ConfigurationException e)
+            {
+                // TODO Auto-generated catch block
+                log.error(e.getMessage());
+            }
         }
     }
 
-- 
2.7.4