diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 1845735922..23a15efd0b 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,7 @@
+2005-06-07  Mark Mitchell  <mark@codesourcery.com>
+
+	* opncls.c (bfd_fdopenr): Add missing break statements.
+
 2005-06-07  Mark Mitchell  <mark@codesourcery.com>
 
 	* opncls.c (bfd_fopen): New API.
diff --git a/bfd/opncls.c b/bfd/opncls.c
index e6755109c5..74bee5777a 100644
--- a/bfd/opncls.c
+++ b/bfd/opncls.c
@@ -280,9 +280,9 @@ bfd_fdopenr (const char *filename, const char *target, int fd)
   /* (O_ACCMODE) parens are to avoid Ultrix header file bug.  */
   switch (fdflags & (O_ACCMODE))
     {
-    case O_RDONLY: mode = FOPEN_RB;
-    case O_WRONLY: mode = FOPEN_RUB;
-    case O_RDWR:   mode = FOPEN_RUB;
+    case O_RDONLY: mode = FOPEN_RB; break;
+    case O_WRONLY: mode = FOPEN_RUB; break;
+    case O_RDWR:   mode = FOPEN_RUB; break;
     default: abort ();
     }
 #endif