X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C851CF.FF9B0B48@onstor-exch02.onstor.net>; Tue, 8 Jan 2008 01:25:16 -0700
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----_=_NextPart_001_01C851CF.FF9B0B48"
Content-class: urn:content-classes:message
Subject: RE: Need opinion on fixing Coverity defects in open source code
Date: Tue, 8 Jan 2008 01:25:15 -0700
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E077645FA@onstor-exch02.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E077645F5@onstor-exch02.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: Need opinion on fixing Coverity defects in open source code
Thread-Index: AchDr+/YmTyrjetnR/6rBg6KfQwEbwF/rBigAggT2+AAACKZYA==
References: <BB375AF679D4A34E9CA8DFA650E2B04E077645F5@onstor-exch02.onstor.net>
From: "Sudharsan Srinivasan" <sudharsan@onstor.com>
To: "dl-Software" <dl-software@onstor.com>

This is a multi-part message in MIME format.

------_=_NextPart_001_01C851CF.FF9B0B48
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

Also, when we make the fix, should the fix be within #ifdef NFX or can
we change the open source code without any #ifdef. The concern that we
had earlier is that the #ifdef will make it more difficult to merge.

=20

________________________________

From: Sudharsan Srinivasan=20
Sent: Tuesday, January 08, 2008 1:49 PM
To: dl-Software
Subject: Need opinion on fixing Coverity defects in open source code

=20

Hi Team,

As we started fixing Coverity reported defects, we had prioritized
ONStor Coverity bugs over Coverity bugs in open source code. We would
like to know if we have to fix Coverity reported defects in open source.

=20

Regards,

Sudharsan

=20

=20

________________________________

From: Brian DeForest=20
Sent: Saturday, December 29, 2007 5:44 AM
To: Sudharsan Srinivasan
Subject: RE: Pending inputs

=20

2.  Agreed, we determined open source Coverity bugs were lower priority
than ONStor Coverity bugs.  Also, you could send email to dl-software to
request opinions.

=20

________________________________

From: Sudharsan Srinivasan=20
Sent: Friday, December 21, 2007 1:01 AM
To: Brian DeForest
Cc: Sudharsan Srinivasan
Subject: Pending inputs

Hi Brian,

This is just a gentle reminder that I am waiting for inputs for some of
the queries raised. The list below is in order of priority.

=20

2.	Further, many Coverity defects are reported in open source code.
We are not working on these, as a decision was made while we were at
onsite to not to work on open source code.=20

=20

Regards,

Sudharsan


------_=_NextPart_001_01C851CF.FF9B0B48
Content-Type: text/html;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

<html xmlns:v=3D"urn:schemas-microsoft-com:vml" =
xmlns:o=3D"urn:schemas-microsoft-com:office:office" =
xmlns:w=3D"urn:schemas-microsoft-com:office:word" =
xmlns:x=3D"urn:schemas-microsoft-com:office:excel" =
xmlns=3D"http://www.w3.org/TR/REC-html40">

<head>
<meta http-equiv=3DContent-Type content=3D"text/html; =
charset=3Dus-ascii">
<meta name=3DGenerator content=3D"Microsoft Word 11 (filtered medium)">
<!--[if !mso]>
<style>
v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style>
<![endif]-->
<style>
<!--
 /* Font Definitions */
 @font-face
	{font-family:Tahoma;
	panose-1:2 11 6 4 3 5 4 4 2 4;}
 /* Style Definitions */
 p.MsoNormal, li.MsoNormal, div.MsoNormal
	{margin:0in;
	margin-bottom:.0001pt;
	font-size:12.0pt;
	font-family:"Times New Roman";}
a:link, span.MsoHyperlink
	{color:blue;
	text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
	{color:purple;
	text-decoration:underline;}
span.EmailStyle17
	{mso-style-type:personal;
	font-family:Arial;
	color:windowtext;}
span.EmailStyle18
	{mso-style-type:personal;
	font-family:Arial;
	color:navy;}
span.EmailStyle19
	{mso-style-type:personal-reply;
	font-family:Arial;
	color:navy;}
@page Section1
	{size:8.5in 11.0in;
	margin:1.0in 1.25in 1.0in 1.25in;}
div.Section1
	{page:Section1;}
 /* List Definitions */
 @list l0
	{mso-list-id:1514103670;
	mso-list-template-ids:-1039735234;}
@list l0:level1
	{mso-level-start-at:2;
	mso-level-tab-stop:.5in;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level2
	{mso-level-tab-stop:1.0in;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level3
	{mso-level-tab-stop:1.5in;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level4
	{mso-level-tab-stop:2.0in;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level5
	{mso-level-tab-stop:2.5in;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level6
	{mso-level-tab-stop:3.0in;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level7
	{mso-level-tab-stop:3.5in;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level8
	{mso-level-tab-stop:4.0in;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level9
	{mso-level-tab-stop:4.5in;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l1
	{mso-list-id:1898737474;
	mso-list-template-ids:64247618;}
@list l1:level1
	{mso-level-start-at:2;
	mso-level-tab-stop:.5in;
	mso-level-number-position:left;
	text-indent:-.25in;}
ol
	{margin-bottom:0in;}
ul
	{margin-bottom:0in;}
-->
</style>

</head>

<body lang=3DEN-US link=3Dblue vlink=3Dpurple>

<div class=3DSection1>

<p class=3DMsoNormal><font size=3D2 color=3Dnavy face=3DArial><span =
style=3D'font-size:
10.0pt;font-family:Arial;color:navy'>Also, when we make the fix, should =
the fix
be within #ifdef NFX or can we change the open source code without any =
#ifdef. The
concern that we had earlier is that the #ifdef will make it more =
difficult to
merge.<o:p></o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 color=3Dnavy face=3DArial><span =
style=3D'font-size:
10.0pt;font-family:Arial;color:navy'><o:p>&nbsp;</o:p></span></font></p>

<div>

<div class=3DMsoNormal align=3Dcenter style=3D'text-align:center'><font =
size=3D3
face=3D"Times New Roman"><span style=3D'font-size:12.0pt'>

<hr size=3D2 width=3D"100%" align=3Dcenter tabindex=3D-1>

</span></font></div>

<p class=3DMsoNormal><b><font size=3D2 face=3DTahoma><span =
style=3D'font-size:10.0pt;
font-family:Tahoma;font-weight:bold'>From:</span></font></b><font =
size=3D2
face=3DTahoma><span style=3D'font-size:10.0pt;font-family:Tahoma'> =
Sudharsan
Srinivasan <br>
<b><span style=3D'font-weight:bold'>Sent:</span></b> Tuesday, January =
08, 2008
1:49 PM<br>
<b><span style=3D'font-weight:bold'>To:</span></b> dl-Software<br>
<b><span style=3D'font-weight:bold'>Subject:</span></b> Need opinion on =
fixing
Coverity defects in open source code</span></font><o:p></o:p></p>

</div>

<p class=3DMsoNormal><font size=3D3 face=3D"Times New Roman"><span =
style=3D'font-size:
12.0pt'><o:p>&nbsp;</o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'>Hi Team,<o:p></o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'>As we started fixing Coverity reported defects, we =
had
prioritized ONStor Coverity bugs over Coverity bugs in open source code. =
We
would like to know if we have to fix Coverity reported defects in open =
source.<o:p></o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'><o:p>&nbsp;</o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'>Regards,<o:p></o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'>Sudharsan<o:p></o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 color=3Dnavy face=3DArial><span =
style=3D'font-size:
10.0pt;font-family:Arial;color:navy'><o:p>&nbsp;</o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 color=3Dnavy face=3DArial><span =
style=3D'font-size:
10.0pt;font-family:Arial;color:navy'><o:p>&nbsp;</o:p></span></font></p>

<div>

<div class=3DMsoNormal align=3Dcenter style=3D'text-align:center'><font =
size=3D3
face=3D"Times New Roman"><span style=3D'font-size:12.0pt'>

<hr size=3D2 width=3D"100%" align=3Dcenter tabindex=3D-1>

</span></font></div>

<p class=3DMsoNormal><b><font size=3D2 face=3DTahoma><span =
style=3D'font-size:10.0pt;
font-family:Tahoma;font-weight:bold'>From:</span></font></b><font =
size=3D2
face=3DTahoma><span style=3D'font-size:10.0pt;font-family:Tahoma'> Brian =
DeForest <br>
<b><span style=3D'font-weight:bold'>Sent:</span></b> Saturday, December =
29, 2007
5:44 AM<br>
<b><span style=3D'font-weight:bold'>To:</span></b> Sudharsan =
Srinivasan<br>
<b><span style=3D'font-weight:bold'>Subject:</span></b> RE: Pending =
inputs</span></font><o:p></o:p></p>

</div>

<p class=3DMsoNormal><font size=3D3 face=3D"Times New Roman"><span =
style=3D'font-size:
12.0pt'><o:p>&nbsp;</o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 color=3Dblue face=3DArial><span =
style=3D'font-size:
10.0pt;font-family:Arial;color:blue'>2.&nbsp; Agreed, we determined open =
source
Coverity bugs were lower priority than ONStor Coverity bugs.&nbsp; Also, =
you
could send email to dl-software to request =
opinions.</span></font><o:p></o:p></p>

<p class=3DMsoNormal><font size=3D3 face=3D"Times New Roman"><span =
style=3D'font-size:
12.0pt'><o:p>&nbsp;</o:p></span></font></p>

<div class=3DMsoNormal align=3Dcenter style=3D'text-align:center'><font =
size=3D3
face=3D"Times New Roman"><span style=3D'font-size:12.0pt'>

<hr size=3D2 width=3D"100%" align=3Dcenter tabIndex=3D-1>

</span></font></div>

<p class=3DMsoNormal style=3D'margin-bottom:12.0pt'><b><font size=3D2 =
face=3DTahoma><span
style=3D'font-size:10.0pt;font-family:Tahoma;font-weight:bold'>From:</spa=
n></font></b><font
size=3D2 face=3DTahoma><span =
style=3D'font-size:10.0pt;font-family:Tahoma'> Sudharsan
Srinivasan <br>
<b><span style=3D'font-weight:bold'>Sent:</span></b> Friday, December =
21, 2007
1:01 AM<br>
<b><span style=3D'font-weight:bold'>To:</span></b> Brian DeForest<br>
<b><span style=3D'font-weight:bold'>Cc:</span></b> Sudharsan =
Srinivasan<br>
<b><span style=3D'font-weight:bold'>Subject:</span></b> Pending =
inputs</span></font><o:p></o:p></p>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'>Hi Brian,<o:p></o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'>This is just a gentle reminder that I am waiting for =
inputs
for some of the queries raised. The list below is in order of =
priority.<o:p></o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'><o:p>&nbsp;</o:p></span></font></p>

<ol style=3D'margin-top:0in' start=3D2 type=3D1>
 <li class=3DMsoNormal style=3D'mso-list:l0 level1 lfo3'><font size=3D2 =
face=3DArial><span
     style=3D'font-size:10.0pt;font-family:Arial'>Further, many Coverity =
defects
     are reported in open source code. We are not working on these, as a
     decision was made while we were at onsite to not to work on open =
source
     code.</span></font> <font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
     font-family:Arial'><o:p></o:p></span></font></li>
</ol>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'><o:p>&nbsp;</o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'>Regards,<o:p></o:p></span></font></p>

<p class=3DMsoNormal><font size=3D2 face=3DArial><span =
style=3D'font-size:10.0pt;
font-family:Arial'>Sudharsan<o:p></o:p></span></font></p>

</div>

</body>

</html>

------_=_NextPart_001_01C851CF.FF9B0B48--
