[BUG]

Wünsche, Anträge, Fehlermeldungen
re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

[BUG]

Beitrag von re_Look »

Code: Alles auswählen

Index: stringinput.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/widget/stringinput.cpp,v
retrieving revision 1.92
diff -u -r1.92 stringinput.cpp
--- a/stringinput.cpp   6 Dec 2008 16:54:32 -0000       1.92
+++ b/stringinput.cpp   3 Mar 2009 08:09:59 -0000
@@ -802,7 +802,7 @@
        if (pixbuf != NULL)
        {
                frameBuffer->RestoreScreen(x- borderwidth, y- borderwidth, width+ 2* borderwidth, height+ 2* borderwidth, pixbuf);
-               delete pixbuf;
+               delete[] pixbuf;
        }

        return ( res );

GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [BUG]

Beitrag von GetAway »

Hi,

schön das Du Dich intensiv mit diesem Thema beschäftigst.
Da das so aber ziemlich träge abläuft bis es eingecheckt
wird, solltest Du vielleicht mal nach Schreibrechte fragen. :wink:

http://forum.tuxbox-cvs.sourceforge.net ... 18&t=48498
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [BUG]

Beitrag von rhabarber1848 »

GetAway hat geschrieben:Da das so aber ziemlich träge abläuft bis es eingecheckt wird
Ich könnte es sofort einchecken, wenn ich wüßte, ob der Patch ok ist.
Ich habe keine Ahnung von C++... :oops:
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: [BUG]

Beitrag von Gaucho316 »

Ich habe auch nicht so die große Ahnung von C++, aber etwas weiter oben im Code wird mit

Code: Alles auswählen

fb_pixel_t * pixbuf = new fb_pixel_t[...];
ein Array erzeugt, das immer mit delete[] gelöscht werden muss. Ein Objekt dagegen (also kein Array) wird mit delete gelöscht.
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [BUG]

Beitrag von rhabarber1848 »

re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

[BUG] Pugins:

Beitrag von re_Look »

Code: Alles auswählen

Index: fx2/solitair/rle.cpp                                           
===================================================================   
RCS file: /cvs/tuxbox/apps/tuxbox/plugins/fx2/solitair/rle.cpp,v      
retrieving revision 1.1                                               
diff -u -r1.1 rle.cpp                                                 
--- a/fx2/solitair/rle.cpp      25 Aug 2003 13:56:57 -0000      1.1   
+++ b/fx2/solitair/rle.cpp      4 Mar 2009 16:54:52 -0000             
@@ -188,7 +188,7 @@                                                   
        fclose( out );                                                
                                                                      
        Decompress( filename_out );                                   
-                                                                     
+       free(filename_out);                                           
                                                                      
        return 0;                                                     
 }                                                                    
@@ -214,6 +214,7 @@                                                   
        if( cursor == 255 || EOF == filename_out[cursor] )            
        {                                                             
                printf( "Decompress: invalid file format!\n" );       
+               fclose( in );                                         
                return 3;                                             
        }                                                             
                                                                      
@@ -225,6 +226,7 @@                                                   
        if( NULL == out )                                             
        {                                                             
                printf("Cannot create [%s] for output\n", filename_out );
+               fclose( in );                                            
                return 2;                                                
        }                                                                
                                                                         
@@ -237,6 +239,8 @@                                                      
                if( !(byte&128) )                                        
                {                                                        
                        printf("Some error while decompressing!");       
+                       fclose( in );                                    
+                       fclose( out );                                   
                        return 3;                                        
                }                                                        
                                                                         
Index: fx2/tetris/somain.c                                               
===================================================================      
RCS file: /cvs/tuxbox/apps/tuxbox/plugins/fx2/tetris/somain.c,v          
retrieving revision 1.34                                                 
diff -u -r1.34 somain.c                                                  
--- a/fx2/tetris/somain.c       17 Dec 2008 21:29:14 -0000      1.34     
+++ b/fx2/tetris/somain.c       4 Mar 2009 16:54:52 -0000                
@@ -512,6 +512,7 @@                                                      
 #endif                                                                  
                }                                                        
                fclose(fp);                                              
+               free(line);                                              
        }                                                                
                                                                         
        fd = open( GAMESDIR "/tetris.hscore", O_RDONLY );                
Index: vncviewer/rfb/vncauth.c
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/plugins/vncviewer/rfb/vncauth.c,v
retrieving revision 1.2
diff -u -r1.2 vncauth.c
--- a/vncviewer/rfb/vncauth.c   23 Jun 2004 11:15:33 -0000      1.2
+++ b/vncviewer/rfb/vncauth.c   4 Mar 2009 16:54:53 -0000
@@ -96,8 +96,10 @@
     int i, ch;
     unsigned char *passwd = (unsigned char *)malloc(9);

-    if ((fp = fopen(fname,"r")) == NULL) return NULL;
-
+    if ((fp = fopen(fname,"r")) == NULL){
+       free(passwd);
+       return NULL;
+    }
     for (i = 0; i < 8; i++) {
        ch = getc(fp);
        if (ch == EOF) {
ps.GetAway für die paar Zeilen lohnt sich nicht.
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: [BUG]

Beitrag von seife »

Hi re_Look,

nur aus interesse: schaust du den ganzen Code manuell durch, oder benutzt du ein Tool (wenn ja: welches?) um diese bugs zu finden?

Ich habe ja auch schon mal zapit und sectionsd per valgrind "bearbeitet", aber bei Neutrino ist mir das bisher nicht gelungen ;)
re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

Re: [BUG]

Beitrag von re_Look »

Code: Alles auswählen

Index: liblcddisplay/fontrenderer.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/libs/liblcddisplay/fontrenderer.cpp,v
retrieving revision 1.11
diff -u -r1.11 fontrenderer.cpp
--- a/liblcddisplay/fontrenderer.cpp    24 Feb 2009 19:09:06 -0000      1.11
+++ b/liblcddisplay/fontrenderer.cpp    4 Mar 2009 17:07:52 -0000
@@ -133,6 +133,7 @@
        if ((error=FT_New_Face(library, filename, 0, &face)))
        {
                printf(" failed: %i\n", error);
+               delete n;
                return NULL;
        }
        n->filename = strdup(filename);
Manuel und Heute mit cppcheck.
http://cppcheck.wiki.sourceforge.net/
Neutrino und zapit sind auber suber laut cppcheck. ;)
re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

Re: [BUG]

Beitrag von re_Look »

Code: Alles auswählen

Index: flashimagecramfs.cpp                                           
===================================================================   
RCS file: /cvs/tuxbox/apps/misc/libs/libflashimage/flashimagecramfs.cpp,v
retrieving revision 1.5                                                  
diff -u -r1.5 flashimagecramfs.cpp                                       
--- a/flashimagecramfs.cpp      29 Jun 2002 14:31:50 -0000      1.5
+++ b/flashimagecramfs.cpp      4 Mar 2009 17:20:11 -0000
@@ -190,7 +190,7 @@
         if ( S_ISDIR ( CRAMFS_16 ( inode -> mode ) ) )
         {
           unsigned int ret = readdir ( stream, CRAMFS_GET_OFFSET ( inode ) << 2, CRAMFS_24 ( inode -> size ), filename );
-          delete buf;
+          delete [] buf;
           return ret;
         }
         else if ( S_ISREG ( CRAMFS_16 ( inode -> mode ) ) )
@@ -207,7 +207,7 @@

   catch ( ... )
   {
-    delete buf;
+    delete [] buf;
     throw;
   }
 }
@@ -271,11 +271,9 @@
   catch ( ... )
   {
     inflateEnd ( & z );
-    delete buf_in;
-    delete buf_block;
   }

-  delete buf_in;
-  delete buf_block;
+  delete [] buf_in;
+  delete [] buf_block;
 }

Das mit dem "catch ( ... )" soll sich aber jemand anschauen. Ich bin nicht 100% sicher bzw weiss ich nicht was catch macht.
re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

Re: [BUG]

Beitrag von re_Look »

Code: Alles auswählen

===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/driver/rcinput.cpp,v
retrieving revision 1.161
diff -u -r1.161 rcinput.cpp
--- a/rcinput.cpp       18 Feb 2009 17:59:43 -0000      1.161
+++ b/rcinput.cpp       4 Mar 2009 17:32:59 -0000
@@ -856,7 +856,7 @@
                                                                        gettimeofday( &tv, NULL );
                                                                        long long timeNew = tv.tv_usec + tv.tv_sec * 1000000LL;

-                                                                       delete p;
+                                                                       delete [] p;
                                                                        p= new unsigned char[ sizeof(long long) ];
                                                                        *(long long*) p = timeNew - timeOld;

@@ -1105,7 +1105,7 @@
                                                printf("[neutrino] event - unknown initiatorID 0x%x\n",  emsg.initiatorID);
                                        if ( !dont_delete_p )
                                        {
-                                               delete p;
+                                               delete [] p;
                                                p= NULL;
                                        }
                                }
re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

Re: [BUG]

Beitrag von re_Look »

Code: Alles auswählen

Index: fontrenderer.cpp                                                  
===================================================================      
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/driver/fontrenderer.cpp,v 
retrieving revision 1.57                                                 
diff -u -r1.57 fontrenderer.cpp                                          
--- a/fontrenderer.cpp  24 Feb 2009 19:09:13 -0000      1.57             
+++ b/fontrenderer.cpp  4 Mar 2009 17:50:30 -0000                        
@@ -156,6 +156,7 @@                                                      
        if ((error=FT_New_Face(library, filename, 0, &face)))            
        {
                dprintf(DEBUG_NORMAL, "[FONT] adding font %s, failed: %i\n", filename, error);
+               delete n;
                return NULL;
        }
        n->filename = strdup(filename);
re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

Re: [BUG]

Beitrag von re_Look »

Code: Alles auswählen

RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/update.cpp,v       
retrieving revision 1.131                                             
diff -u -r1.131 update.cpp                                            
--- a/update.cpp        24 Jan 2009 15:24:46 -0000      1.131         
+++ b/update.cpp        5 Mar 2009 18:15:53 -0000                     
@@ -652,6 +652,7 @@                                                   
                mtdselector->addItem(new CMenuForwarderNonLocalized(mtdInfo->getMTDName(x).c_str(), true, NULL, this, sActionKey));                                                                                                                         
        }                                                                                                                     
        mtdselector->exec(NULL,"");                                                                                           
+       delete mtdselector;                                                                                                   
 }                                                                                                                            
                                                                                                                              
 void CFlashExpert::showFileSelector(const std::string & actionkey)                                                           
@@ -687,6 +688,7 @@                                                                                                           
                free(namelist);                                                                                               
        }                                                                                                                     
        fileselector->exec(NULL,"");                                                                                          
+       delete fileselector;
 }


Index: upnpbrowser.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/upnpbrowser.cpp,v
retrieving revision 1.8
diff -u -r1.8 upnpbrowser.cpp
--- a/upnpbrowser.cpp   5 Dec 2008 22:06:19 -0000       1.8
+++ b/upnpbrowser.cpp   5 Mar 2009 18:15:55 -0000
@@ -231,9 +231,10 @@
        parser = new XMLTreeParser("UTF-8");
        parser->Parse(result.c_str(), result.size(), 1);
        root=parser->RootNode();
-       if (!root)
+       if (!root){
+               delete parser;
                return NULL;
-
+       }
        entries = new std::vector<UPnPEntry>;

        for (node=root->GetChild(); node; node=node->GetNext())
@@ -378,6 +379,7 @@
                        entries->push_back(entry);
                }
        }
+       delete parser;
        return entries;
 }

re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

Re: [BUG]

Beitrag von re_Look »

Code: Alles auswählen

RCS file: /cvs/tuxbox/apps/misc/libs/libmpegtools/ctools.c,v
retrieving revision 1.3
diff -u -r1.3 ctools.c
--- a/ctools.c  29 Apr 2003 06:51:57 -0000      1.3
+++ b/ctools.c  5 Mar 2009 18:33:12 -0000
@@ -596,7 +596,10 @@

        if (p->length >0){
                buf = (uint8_t *) malloc(p->length);
-               if((neof = save_read(f,buf,p->length))< p->length) return -1;
+               if((neof = save_read(f,buf,p->length))< p->length){
+                   if(buf!=NULL) free(buf);
+                   return -1;
+                }
                cread_pes((char *)buf,p);
                free(buf);
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [BUG]

Beitrag von rhabarber1848 »

Hier die gesammelten Werke: re_look.diff
Ich teste gerade die Kompilierbarkeit.
dbt
Administrator
Beiträge: 2675
Registriert: Donnerstag 28. September 2006, 19:18

Re: [BUG]

Beitrag von dbt »

Leaks stopfen ist immer gut :D. Compile ist grade durchgelaufen :wink:
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [BUG]

Beitrag von rhabarber1848 »

rhabarber1848 hat geschrieben:Hier die gesammelten Werke: re_look.diff
Alles kompilierte und ist bereits im CVS.
PauleFoul
Wissender
Wissender
Beiträge: 1839
Registriert: Sonntag 17. August 2003, 01:39

Re: [BUG]

Beitrag von PauleFoul »

Tolle Arbeit!! Finde ich klasse das hier so intensiv an den Mem-Leaks gearbeitet wird...

Gruß
____Paule
re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

Re: [BUG]

Beitrag von re_Look »

strdup nutzt malloc, deswegen free und nicht delete.

Code: Alles auswählen

Index: fontrenderer.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/libs/liblcddisplay/fontrenderer.cpp,v
retrieving revision 1.13
diff -u -r1.13 fontrenderer.cpp
--- a/fontrenderer.cpp  9 Mar 2009 09:14:15 -0000       1.13
+++ b/fontrenderer.cpp  9 Mar 2009 14:25:59 -0000
@@ -150,9 +150,9 @@

 LcdFontRenderClass::fontListEntry::~fontListEntry()
 {
-       delete[] filename;
-       delete[] family;
-       delete[] style;
+       free(filename);
+       free(family);
+       free(style);
 }

 LcdFont *LcdFontRenderClass::getFont(const char *family, const char *style, int size)

Code: Alles auswählen

Index: fontrenderer.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/driver/fontrenderer.cpp,v
retrieving revision 1.59
diff -u -r1.59 fontrenderer.cpp
--- a/fontrenderer.cpp  9 Mar 2009 09:07:16 -0000       1.59
+++ b/fontrenderer.cpp  9 Mar 2009 14:28:27 -0000
@@ -171,9 +171,9 @@

 FBFontRenderClass::fontListEntry::~fontListEntry()
 {
-       delete[] filename;
-       delete[] family;
-       delete[] style;
+       free(filename);
+       free(family);
+       free(style);
 }

 Font *FBFontRenderClass::getFont(const char * const family, const char * const style, int size)
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [BUG]

Beitrag von rhabarber1848 »

re_Look hat geschrieben:strdup nutzt malloc, deswegen free und nicht delete.
committed to CVS
re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

Re: [BUG] Memory leak & Resource leak

Beitrag von re_Look »

Code: Alles auswählen

RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/themes.cpp,v
retrieving revision 1.8
diff -u -r1.8 themes.cpp
--- a/themes.cpp        27 Jan 2009 02:24:19 -0000      1.8
+++ b/themes.cpp        26 Mar 2009 18:02:52 -0000
@@ -160,6 +160,7 @@
                themes.addItem(GenericMenuSeparatorLine);
                themes.addItem(m1);
        } else {
+               delete m1;
                printf("[neutrino theme] error accessing %s\n", USERDIR);
        }

Code: Alles auswählen

Index: tuxboxapi/controlapi.cpp                                         
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/nhttpd/tuxboxapi/controlapi.cpp,v
retrieving revision 1.17
diff -u -r1.17 controlapi.cpp
--- a/tuxboxapi/controlapi.cpp  11 Mar 2009 20:42:15 -0000      1.17
+++ b/tuxboxapi/controlapi.cpp  26 Mar 2009 18:05:28 -0000
@@ -1950,6 +1950,7 @@
                        CConfigFile *Config = new CConfigFile(',');
                        Config->loadConfig(NEUTRINO_CONFIGFILE);
                        _rec_dir = Config->getString("recording_dir_0", "/mnt/filme");
+                       delete Config;
                }
                if(changeApids)
                        eventinfo.apids = apids;
Index: tuxboxapi/neutrinoyparser.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/nhttpd/tuxboxapi/neutrinoyparser.cpp,v
retrieving revision 1.5
diff -u -r1.5 neutrinoyparser.cpp
--- a/tuxboxapi/neutrinoyparser.cpp     18 Feb 2009 17:47:26 -0000      1.5
+++ b/tuxboxapi/neutrinoyparser.cpp     26 Mar 2009 18:05:28 -0000
@@ -970,6 +970,7 @@
                CConfigFile *Config = new CConfigFile(',');
                Config->loadConfig(NEUTRINO_CONFIGFILE);
                hh->ParamList["timer_recordingDir"] = Config->getString("network_nfs_recordingdir", "/mnt/filme");
+               delete Config;
        }
        hh->ParamList["standby"] = (cmd == "new")? "0" : ((timer.standby_on)?"1":"0");
        hh->ParamList["message"] = (cmd == "new")? "" : timer.message;

Code: Alles auswählen

Index: pictureviewer/png.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/driver/pictureviewer/png.cpp,v
retrieving revision 1.13
diff -u -r1.13 png.cpp
--- a/pictureviewer/png.cpp     24 Jun 2007 11:40:49 -0000      1.13
+++ b/pictureviewer/png.cpp     26 Mar 2009 18:07:12 -0000
@@ -39,7 +39,10 @@
        if(!(fh=fopen(name,"rb")))      return(FH_ERROR_FILE);

        png_ptr = png_create_read_struct(PNG_LIBPNG_VER_STRING,NULL,NULL,NULL);
-       if(png_ptr == NULL) return(FH_ERROR_FORMAT);
+       if(png_ptr == NULL){
+           fclose(fh);
+           return(FH_ERROR_FORMAT);
+       }
        info_ptr = png_create_info_struct(png_ptr);
        if(info_ptr == NULL)
        {
@@ -119,7 +122,10 @@
        if(!(fh=fopen(name,"rb")))      return(FH_ERROR_FILE);

        png_ptr = png_create_read_struct(PNG_LIBPNG_VER_STRING,NULL,NULL,NULL);
-       if(png_ptr == NULL) return(FH_ERROR_FORMAT);
+       if(png_ptr == NULL){
+         fclose(fh);
+         return(FH_ERROR_FORMAT);
+       }
        info_ptr = png_create_info_struct(png_ptr);
        if(info_ptr == NULL)
        {

Code: Alles auswählen

RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/driver/pictureviewer/jpeg.cpp,v    
retrieving revision 1.10                                                          
diff -u -r1.10 jpeg.cpp                                                           
--- a/jpeg.cpp  24 Jun 2007 11:40:49 -0000      1.10                              
+++ b/jpeg.cpp  26 Mar 2009 18:13:15 -0000                                        
@@ -64,7 +64,7 @@                                                                 
        struct pic_data pd;                                                       
                                                                                  
        strncpy(path, filename, PICV_CLIENT_SERVER_PATHLEN-1);                    
-       path[PICV_CLIENT_SERVER_PATHLEN]=0;                                       
+       path[PICV_CLIENT_SERVER_PATHLEN-1]=0;                                     

        dbout("fh_jpeg_load_via_server (%s/%d/%d) {\n",basename(filename),x,y);
        if ((s=socket(AF_INET, SOCK_STREAM, IPPROTO_TCP))==-1)
re_Look
Interessierter
Interessierter
Beiträge: 47
Registriert: Mittwoch 10. Oktober 2007, 07:20

Re: [BUG]

Beitrag von re_Look »

more memleak fixes due to wrong rcinput data delete()
Das ist auch noch in gui/scan.cpp und gui/motorcontrol.cpp und noch ein in neutrino.cpp.
CNeutrinoApp::handleMsg
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: [BUG]

Beitrag von seife »

Das Problem ist, dass nicht immer ganz klar ist, wo die Sachen alloziert wurden. Die meisten (aber anscheinend nicht alle) werden in rcinput.cpp:761 alloziert.

Tatsächlich fixe ich üblicherweise nur die, die ich mit Valgrind reproduziert habe (allerdings ist das schwierig bei Events, die selten auftreten), weil es nur durch Anschauen des Codes schwer ist, das genau nachzuvollziehen.

Wenn ein "delete" anstelle eines "delete[]" gemacht wird, leaken wir ein paar Bytes. Andersrum segfaulted es üblicherweise. Ich bevorzuge doch das Erste.

Obwohl - der Segfault zeigt sehr direkt, dass da was faul ist ;-)
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [BUG] Memory leak & Resource leak

Beitrag von rhabarber1848 »

re_Look hat geschrieben:RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/themes.cpp,v
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/nhttpd/tuxboxapi/controlapi.cpp,v
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/nhttpd/tuxboxapi/neutrinoyparser.cpp,v
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/driver/pictureviewer/png.cpp,v
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/driver/pictureviewer/jpeg.cpp,v
committed to CVS
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [BUG]

Beitrag von GetAway »

Code: Alles auswählen

diff -u -p -r1.47 colorchooser.cpp
--- a/apps/tuxbox/neutrino/src/gui/widget/colorchooser.cpp	9 Nov 2009 21:38:27 -0000	1.47
+++ b/apps/tuxbox/neutrino/src/gui/widget/colorchooser.cpp	14 Dec 2009 22:38:50 -0000
@@ -191,6 +191,7 @@ int CColorChooser::exec(CMenuTarget* par
 				*value[VALUE_B] = b_alt;
 				if (value[VALUE_ALPHA])
 					*value[VALUE_ALPHA] = a_alt;
+				break;
 
 			case CRCInput::RC_timeout:
 				loop = false;
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [BUG]

Beitrag von rhabarber1848 »

Welches Problem wird mit diesem Patch behoben?