# HG changeset patch
# User Ulf Zibis
# Date 1460541927 -7200
#      Wed Apr 13 12:05:27 2016 +0200
# Node ID 8fe4e9e461347828d58403298cc4dd1ab02487cd
# Parent  b6131e088960185594915d3fc9af2d05aa3a8b57
Added missing break, commented superfluous ones.

diff --git a/src/secaudit.c b/src/secaudit.c
--- a/src/secaudit.c
+++ b/src/secaudit.c
@@ -1432,8 +1432,8 @@
 	auth = get6h(attr,off+2);
 	first = get4l(attr,off+8);
 	known = FALSE;
-	if ((attr[off] == 1) /* revision */
-	     && (auth < 100))
+	if (attr[off] == 1) /* revision */
+			&& (auth < 100)) /* switch allows only int values */
 		switch (cnt) {
 		case 0 : /* no level (error) */
 			break;
@@ -1460,7 +1460,7 @@
 				case 1 :
 					known = TRUE;
 					printf("%*cCreator group SID\n",-level,marker);
-					break;
+					break; /* superfluous break */
 				}
 				break;
 			case 5 :
@@ -1504,9 +1504,9 @@
 				case 18 :
 					known = TRUE;
 					printf("%*cNT System SID\n",-level,marker);
-					break;
+					break; /* superfluous break */
 				}
-				break;
+				break; /* superfluous break */
 			}
 			break;
 		case 2 : /* double level */
@@ -1527,11 +1527,12 @@
 						break;
 					default :
 						printf("%*cSome domain SID\n",-level,marker);
-						break;
+						break; /* superfluous break */
 					}
 				}
-				break;
+				break; /* superfluous break */
 			}
+			break;
 		default : /* three levels or more */
 			second = get4l(attr,off+12);
 			last = get4l(attr,off+4+4*cnt);
@@ -1540,28 +1541,29 @@
 				if (first == 21) {
 					known = TRUE;
 					switch (last) {
-						case 500 :
-							printf("%*cSystem admin SID\n",-level,marker);
-							break;
-						case 501 :
-							printf("%*cGuest SID\n",-level,marker);
-							break;
-						case 512 :
-							printf("%*cLocal admins SID\n",-level,marker);
-							break;
-						case 513 :
-							printf("%*cLocal users SID\n",-level,marker);
-							break;
-						case 514 :
-							printf("%*cLocal guests SID\n",-level,marker);
-							break;
-						default :
-							printf("%*cLocal user-%lu SID\n",-level,marker,last);
-							break;
+					case 500 :
+						printf("%*cSystem admin SID\n",-level,marker);
+						break;
+					case 501 :
+						printf("%*cGuest SID\n",-level,marker);
+						break;
+					case 512 :
+						printf("%*cLocal admins SID\n",-level,marker);
+						break;
+					case 513 :
+						printf("%*cLocal users SID\n",-level,marker);
+						break;
+					case 514 :
+						printf("%*cLocal guests SID\n",-level,marker);
+						break;
+					default :
+						printf("%*cLocal user-%lu SID\n",-level,marker,last);
+						break; /* superfluous break */
 					}
 				}
-				break;
+				break; /* superfluous break */
 			}
+			break; /* superfluous break */
 		}
 	if (!known)
 		printf("%*cUnknown SID\n",-level,marker);