[Spice-devel] [PATCH] fix tmp directory removal

Peter Hatina phatina at redhat.com
Tue Apr 9 06:05:16 PDT 2013


Hi,

of course, it is not. That's why I put there a rhbz link.

On 04/09/2013 02:20 PM, Christophe Fergeau wrote:
> On Tue, Apr 09, 2013 at 12:16:38PM +0200, Peter Hatina wrote:
>> Hi,
>> 
>> this should fix
>> https://bugzilla.redhat.com/show_bug.cgi?id=882339

^^

>> 
>> Are we ok with this short diff?
> 
> This patch is not against git master if I'm not mistaken (the
> unlink() are no longer there, nor the ControllerWaitHelper method)
> 
> Christophe
> 
>> 
>> --- SpiceXPI/src/plugin/plugin.cpp | 2 ++ 1 file changed, 2
>> insertions(+)
>> 
>> diff --git a/SpiceXPI/src/plugin/plugin.cpp
>> b/SpiceXPI/src/plugin/plugin.cpp index 4a88e45..d5ca720 100644 
>> --- a/SpiceXPI/src/plugin/plugin.cpp +++
>> b/SpiceXPI/src/plugin/plugin.cpp @@ -996,7 +996,9 @@ void
>> *nsPluginInstance::ControllerWaitHelper(void *opaque) 
>> fake_this->m_external_controller.Disconnect(); 
>> fake_this->m_pid_controller = -1; 
>> unlink(fake_this->m_trust_store_file.c_str()); +
>> unlink(fake_this->m_foreign_file.c_str()); 
>> fake_this->m_trust_store_file.clear(); +
>> fake_this->m_foreign_file.clear(); return NULL; }
>> 
>> -- Peter Hatina EMEA ENG-Base Operating Systems Red Hat Czech,
>> Brno
>> 
>> _______________________________________________ Spice-devel
>> mailing list Spice-devel at lists.freedesktop.org 
>> http://lists.freedesktop.org/mailman/listinfo/spice-devel


-- 
Peter Hatina
EMEA ENG-Base Operating Systems
Red Hat Czech, Brno


More information about the Spice-devel mailing list