Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 343561

Summary: Fix an openssh related issue
Product: [Tools] PTP Reporter: rong li <ronglli>
Component: SCIAssignee: rong li <ronglli>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: tuhongj
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
Submit the patch to fix an openssh related issue. g.watson: iplog+

Description rong li CLA 2011-04-21 10:50:15 EDT 
Build Identifier: 

Fix an opensssh related issue.
Change the log level of "Wrong value of SCI_ENABLE_SSHAUTH" to be log_warn.
Update the lines to new the struct.


Reproducible: Always
Comment 1 rong li CLA 2011-04-21 10:52:18 EDT 
Created attachment 193841 [details]
Submit the patch to fix an openssh related issue.

Submit the patch to fix an openssh related issue.
Comment 2 Tu Hong Jun CLA 2011-04-21 22:20:44 EDT 
Would you mind to explain what is the issue clearly. What do you mean "Wrong value of SCI_ENABLE_SSHAUTH"? I read the patch, it seems there are two major changes
1. Add the ifdef of INSTALL_PATH
2. Fix a memory leak issue
Comment 3 Tu Hong Jun CLA 2011-04-22 00:58:08 EDT 
Have applied patch
Comment 4 rong li CLA 2011-04-22 04:59:34 EDT 
Thanks for your comments.

The "Wrong value of SCI_ENABLE_SSHAUTH" means users set an wrong value to the environment variable "SCI_ENABLE_SSHAUTH"(the correct value should be "yes" or "no"). If users does not set the config file or set the wrong value, the default value will be used. So we updated the log level to be log_warn, to warn the users it will use the default value(it will not lead to SCI errors). 

Besides, we updated the codes about newing the struct to fix a memory leak issue.
Comment 5 rong li CLA 2012-11-06 05:01:24 EST 
This codes have been committed into the branch.