OSDN Git Service

Fix the ISurfaceComposer onTransact switch.
authorJamie Gennis <jgennis@google.com>
Mon, 20 Dec 2010 19:05:18 +0000 (11:05 -0800)
committerMathias Agopian <mathias@google.com>
Mon, 17 Jan 2011 01:34:07 +0000 (17:34 -0800)
Two of the cases in the ISurfaceComposer onTransact switch statement
were missing 'break' statements at the end, and would fall through to
the next case block. This change adds those break statements.

Change-Id: I6dcc84263d3ea03d94612c667103283846b3dee1

libs/surfaceflinger_client/ISurfaceComposer.cpp

index 969ee79..a2a5455 100644 (file)
@@ -247,13 +247,13 @@ status_t BnSurfaceComposer::onTransact(
             int32_t mode = data.readInt32();
             status_t res = turnElectronBeamOff(mode);
             reply->writeInt32(res);
-        }
+        } break;
         case TURN_ELECTRON_BEAM_ON: {
             CHECK_INTERFACE(ISurfaceComposer, data, reply);
             int32_t mode = data.readInt32();
             status_t res = turnElectronBeamOn(mode);
             reply->writeInt32(res);
-        }
+        } break;
         default:
             return BBinder::onTransact(code, data, reply, flags);
     }