Re: *scanf %a, patches...

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Dec 02, 2012 at 08:58:31PM +0100, Juergen Lock wrote:
> On Sun, Dec 02, 2012 at 02:19:44PM +0100, Klaus Schmidinger wrote:
> > On 01.12.2012 21:14, Juergen Lock wrote:
> > > In article <50BA09CC.2040104@xxxxxxxxxxxxxx> you write:
> > >> Am 30.11.2012 11:32, schrieb Gerald Dachs:
> > >>> Am 2012-11-30 10:17, schrieb Lars Hanisch:

> [...]  And the tools.c patch for the
> cCharSetConv destructor looks like an actual bug, iconv_close()
> shouldn't be called with (iconv_t)-1.
> 
Ah sorry that patch isn't in ports yet, and now that I think of it
it only occured when testing the epgfixer plugin which doesn't quite
work properly yet anyway.  But I guess it wouldn't hurt applying
anyway so here it is:

--- tools.c.orig
+++ tools.c
@@ -842,7 +842,8 @@ cCharSetConv::cCharSetConv(const char *F
 cCharSetConv::~cCharSetConv()
 {
   free(result);
-  iconv_close(cd);
+  if (cd != (iconv_t)-1)
+     iconv_close(cd);
 }
 
 void cCharSetConv::SetSystemCharacterTable(const char *CharacterTable)

 Thanx again, :)
	Juergen

_______________________________________________
vdr mailing list
vdr@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


[Index of Archives]     [Linux Media]     [Asterisk]     [DCCP]     [Netdev]     [Xorg]     [Util Linux NG]     [Xfree86]     [Big List of Linux Books]     [Fedora Users]     [Fedora Women]     [ALSA Devel]     [Linux USB]

  Powered by Linux