[tyndur-devel] p ohne v

[Kevin Wolf <kevin@xxxxxxxxxx>]

Nachdem ein tolles neues Spielzeug sogar Ergebnisse gebracht hat,  
poste ich die mal hierhin, damit sie nicht verloren gehen. Der  
folgende spatch...

@@ @@
 p();
 ...
(
 v();
 ...
 return ...;
|
+v();
 return ...;
)

...hat folgende Stellen gefunden, in denen ein v(); fehlt. Unbesehen  
direkt als Patch nehmen sollte man das wohl nicht, aber bei  
Gelegenheit könnte man die Stellen fixen.

diff -u -p a/dma/dma.c b/dma/dma.c
--- a/dma/dma.c 2010-07-30 08:52:11.000000000 +0200
+++ b/dma/dma.c 2010-08-23 14:55:17.000000000 +0200
@@ -137,8 +137,10 @@ bool setup_dma(uint8_t channel, uint8_t

     // Mehrere Zugriffe auf einen DMA-Kanal? Is nich.
     // TODO Evtl. warten, bis der DMA-Kanal frei wird?
-    if (dma_transfers[channel].used)
+    if (dma_transfers[channel].used) {
+        v();
         return false;
+    }

     // Wenn der Kanal schon die richtige Gr��e hat,  
wiederverawenden
     dma_mem_ptr_t dma_ptr;
diff -u -p a/cdi/lib/fs/lostio_if.c b/cdi/lib/fs/lostio_if.c
--- a/cdi/lib/fs/lostio_if.c 2010-07-30 08:52:11.000000000 +0200
+++ b/cdi/lib/fs/lostio_if.c 2010-08-23 14:55:18.000000000 +0200
@@ -474,6 +474,7 @@ int lostio_seek_handler(lostio_filehandl
             break;

         default:
+            v();
             return -1;
     }

diff -u -p a/lib/posix/wait.c b/lib/posix/wait.c
--- a/lib/posix/wait.c 2010-07-30 08:52:11.000000000 +0200
+++ b/lib/posix/wait.c 2010-08-23 14:55:20.000000000 +0200
@@ -88,6 +88,7 @@ void wait_child_add(pid_t pid)
     // Es kann sein, dass der Prozess schon terminiert hat, wenn  
diese funktion
     // aufgerufen wird!
     if (wait_child != NULL) {
+        v();
         return;
     }

diff -u -p a/fat/file.c b/fat/file.c
--- a/fat/file.c 2010-07-30 08:52:11.000000000 +0200
+++ b/fat/file.c 2010-08-23 14:55:20.000000000 +0200
@@ -166,6 +166,7 @@ size_t fat_file_write_handler(lostio_fil
         res_info->cluster = fat_allocate_cluster(filehandle);
         if (res_info->cluster == 0) {
             puts("fat: Keine freien Cluster mehr vorhanden!");
+            v();
             return 0;
         }
         fat_set_fat_entry(filehandle, res_info->cluster, 0xfff);
@@ -184,6 +185,7 @@ size_t fat_file_write_handler(lostio_fil
     if (!fat_save_clusters(filehandle, res_info->cluster, res_info->data,
         res_info->size))
     {
+        v();
         return 0;
     }

@@ -215,6 +217,7 @@ int fat_file_seek_handler(lostio_filehan
             {
                 int newpos = offset;
                 if ((filehandle->flags & IO_OPEN_MODE_WRITE) == 0) {
+                    v();
                     return -1;
                 } else {
                     filehandle->pos = filehandle->node->size;
@@ -225,6 +228,7 @@ int fat_file_seek_handler(lostio_filehan
                     free(data);

                     if (filehandle->pos != filehandle->pos) {
+                        v();
                         return -1;
                     }
                 }
@@ -241,6 +245,7 @@ int fat_file_seek_handler(lostio_filehan
                 int newpos = offset + filehandle->pos;

                 if ((filehandle->flags & IO_OPEN_MODE_WRITE) == 0) {
+                    v();
                     return -1;
                 } else {
                     filehandle->pos = filehandle->node->size;
@@ -251,6 +256,7 @@ int fat_file_seek_handler(lostio_filehan
                     free(data);

                     if (filehandle->pos != filehandle->pos) {
+                        v();
                         return -1;
                     }
                 }