<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:\5B8B\4F53;
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:\5B8B\4F53;
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:"\@\5B8B\4F53";
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:Consolas;
panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
margin-bottom:.0001pt;
text-align:justify;
text-justify:inter-ideograph;
font-size:10.5pt;
font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-compose;
font-family:"Calibri","sans-serif";
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri","sans-serif";}
/* Page Definitions */
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 90.0pt 72.0pt 90.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="ZH-CN" link="blue" vlink="purple" style="text-justify-trim:punctuation">
<div class="WordSection1">
<p class="MsoNormal"><span lang="EN-US">Hi,<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> The following is the current code to release connection in libvirt.<o:p></o:p></span></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:#7F0055">int</span></i></b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">virConnectClose</span></i></b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">(</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:#005032">virConnectPtr</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">
conn)</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">{</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">…</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p> </o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">
</span></i><b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:#7F0055">if</span></i></b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> (!VIR_IS_CONNECT(conn)) {</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> virLibConnError(</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:#0000C0">VIR_ERR_INVALID_CONN</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">,
__FUNCTION__);</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">
</span></i><b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:#7F0055">goto</span></i></b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> error;</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> }</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas">…<o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">error:</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> virDispatchError(<u>NULL</u>);</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal" align="left" style="text-align:left;text-autospace:none"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">
</span></i><b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:#7F0055">return</span></i></b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> ret;</span></i><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas"><o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">}<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">Now if the cable is unplugged and the application call virConnectClose to release connection, the code will enter into the error procedure, the connection<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">Can’t be released. I have changed the following two parts to fix this issue. Please give your comments:<o:p></o:p></span></i></p>
<p class="MsoNormal"><b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">Changed Code1:<o:p></o:p></span></i></b></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">int<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">virConnectClose(virConnectPtr conn)<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">{<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">…<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">+ if(NULL == conn) {<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">+ return 0;<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">+ }<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">…<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">- if (!VIR_IS_CONNECT(conn)) {<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">- virLibConnError(VIR_ERR_INVALID_CONN, __FUNCTION__);<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">- goto error;<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">- }<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">…<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">error:<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> virDispatchError(NULL);<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> return ret;<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">}<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></i></p>
<p class="MsoNormal"><b><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">Changed Code2:<o:p></o:p></span></i></b></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">int<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">virUnrefConnect(virConnectPtr conn) {<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">…<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">+ if(NULL == conn) {<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">+ return 0;<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">+ }<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">- if ((!VIR_IS_CONNECT(conn))) {<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">- virLibConnError(VIR_ERR_INVALID_ARG, _("no connection"));<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">- return -1;<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">- }<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">…<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">}<o:p></o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></i></p>
<p class="MsoNormal"><i><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></i></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">For libvirt java, there are similar issue. I have changed code as following in Collect.java. Please also give your comments.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> public int close() throws LibvirtException {<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> int success = 0;<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> if (VCP != null) {<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">+ try {<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> success = libvirt.virConnectClose(VCP);<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> processError();<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">+ }<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">+ finally {<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> // If leave an invalid pointer dangling around JVM crashes and burns<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> // if someone tries to call a method on us<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> // We rely on the underlying libvirt error handling to detect that<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> // it's called with a null virConnectPointer<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> VCP = null;
<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> + }<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> }<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"> return success;<o:p></o:p></span></p>
<p class="MsoNormal" style="text-indent:19.2pt"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">}<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">B.R.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:black">Benjamin Wang<o:p></o:p></span></p>
</div>
</body>
</html>