diff --git a/src/apps/mail/MailWindow.cpp b/src/apps/mail/MailWindow.cpp
index 731eb582fe..2fbe312f99 100644
--- a/src/apps/mail/MailWindow.cpp
+++ b/src/apps/mail/MailWindow.cpp
@@ -2537,7 +2537,7 @@ TMailWindow::SaveAsDraft()
 				}
 
 				// Cache the ref
-				if (fRef == NULL);
+				if (fRef == NULL)
 					fRef = new entry_ref;
 				BEntry entry(&dir, fileName);
 				entry.GetRef(fRef);
diff --git a/src/kits/interface/ChannelControl.cpp b/src/kits/interface/ChannelControl.cpp
index 1ffa878820..39170cddf3 100644
--- a/src/kits/interface/ChannelControl.cpp
+++ b/src/kits/interface/ChannelControl.cpp
@@ -317,7 +317,7 @@ BChannelControl::SetValue(int32 value)
 	if (value > fChannelMax[fCurrentChannel])
 		value = fChannelMax[fCurrentChannel];
 	
-	if (value < fChannelMin[fCurrentChannel]);
+	if (value < fChannelMin[fCurrentChannel])
 		value = fChannelMin[fCurrentChannel];
 		
 	if (value != fChannelValues[fCurrentChannel]) {
diff --git a/src/servers/app/CursorSet.cpp b/src/servers/app/CursorSet.cpp
index ddf417c7b0..b676b83714 100644
--- a/src/servers/app/CursorSet.cpp
+++ b/src/servers/app/CursorSet.cpp
@@ -164,7 +164,7 @@ CursorSet::RemoveCursor(BCursorID which)
 status_t
 CursorSet::FindCursor(BCursorID which, BBitmap **cursor, BPoint *hotspot)
 {
-	if (!cursor || !hotspot);
+	if (!cursor || !hotspot)
 		return B_BAD_VALUE;
 
 	BMessage msg;