Build check errors found: 2013-01-05

postmaster at open-mesh.org postmaster at open-mesh.org
Sat Jan 5 00:51:04 CET 2013


Name of failed tests
====================

checkpatch master bridge_loop_avoidance.c
checkpatch master icmp_socket.c
checkpatch master soft-interface.c
checkpatch master vis.c
checkpatch next bat_iv_ogm.c
checkpatch next bridge_loop_avoidance.c
checkpatch next debugfs.c
checkpatch next icmp_socket.c
checkpatch next originator.c
checkpatch next routing.c
checkpatch next send.c
checkpatch next soft-interface.c
checkpatch next translation-table.c
checkpatch next unicast.c
checkpatch next vis.c

Output of different failed tests
================================

checkpatch master bridge_loop_avoidance.c:
>>>>>>>>
WARNING: Avoid CamelCase: <vhdr->h_vlan_TCI>
#874: FILE: bridge_loop_avoidance.c:874:
+		vid = ntohs(vhdr->h_vlan_TCI) & VLAN_VID_MASK;

total: 0 errors, 1 warnings, 0 checks, 1699 lines checked

checkpatch master soft-interface.c:
>>>>>>>>
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
WARNING: Avoid CamelCase: <vhdr->h_vlan_TCI>
#170: FILE: soft-interface.c:170:
+		vid = ntohs(vhdr->h_vlan_TCI) & VLAN_VID_MASK;

total: 0 errors, 1 warnings, 0 checks, 667 lines checked

checkpatch next bat_iv_ogm.c:
>>>>>>>>
CHECK: Blank lines aren't necessary after an open brace '{'
#186: FILE: bat_iv_ogm.c:186:
+					 batadv_ogm_packet->tt_num_changes)) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#264: FILE: bat_iv_ogm.c:264:
+	    (forw_packet->own && (forw_packet->if_incoming != primary_if))) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#328: FILE: bat_iv_ogm.c:328:
+	    (aggregated_bytes <= BATADV_MAX_AGGREGATION_BYTES)) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#818: FILE: bat_iv_ogm.c:818:
+				 &orig_neigh_node->neigh_list, list) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#952: FILE: bat_iv_ogm.c:952:
+				 &orig_node->neigh_list, list) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#1226: FILE: bat_iv_ogm.c:1226:
+	if (is_single_hop_neigh) {
+

total: 0 errors, 0 warnings, 6 checks, 1349 lines checked

checkpatch next bridge_loop_avoidance.c:
>>>>>>>>
CHECK: Blank lines aren't necessary after an open brace '{'
#238: FILE: bridge_loop_avoidance.c:238:
+					  head, hash_entry) {
+

CHECK: Blank lines aren't necessary before a close brace '}'
#342: FILE: bridge_loop_avoidance.c:342:
+
+	}

CHECK: Blank lines aren't necessary before a close brace '}'
#543: FILE: bridge_loop_avoidance.c:543:
+
+}

CHECK: Blank lines aren't necessary before a close brace '}'
#602: FILE: bridge_loop_avoidance.c:602:
+
+	}

WARNING: Avoid CamelCase: <vhdr->h_vlan_TCI>
#875: FILE: bridge_loop_avoidance.c:875:
+		vid = ntohs(vhdr->h_vlan_TCI) & VLAN_VID_MASK;

total: 0 errors, 1 warnings, 4 checks, 1706 lines checked

checkpatch next debugfs.c:
>>>>>>>>
CHECK: Blank lines aren't necessary before a close brace '}'
#168: FILE: debugfs.c:168:
+
+	}

WARNING: debugfs_remove_recursive(NULL) is safe this check is probably not required
#401: FILE: debugfs.c:401:
+	if (batadv_debugfs) {
+		debugfs_remove_recursive(batadv_debugfs);

total: 0 errors, 1 warnings, 1 checks, 459 lines checked

checkpatch next icmp_socket.c:
>>>>>>>>
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.

checkpatch next originator.c:
>>>>>>>>
CHECK: Blank lines aren't necessary after an open brace '{'
#181: FILE: originator.c:181:
+					  head, hash_entry) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#288: FILE: originator.c:288:
+				  &orig_node->neigh_list, list) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#296: FILE: originator.c:296:
+		    (if_incoming->if_status == BATADV_IF_TO_BE_REMOVED)) {
+

total: 0 errors, 0 warnings, 3 checks, 645 lines checked

checkpatch next routing.c:
>>>>>>>>
CHECK: Blank lines aren't necessary after an open brace '{'
#83: FILE: routing.c:83:
+	} else if ((!curr_router) && (neigh_node)) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#175: FILE: routing.c:175:
+				 &orig_node->neigh_list, list) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#839: FILE: routing.c:839:
+				       neigh_node->if_incoming->net_dev->mtu)) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#1106: FILE: routing.c:1106:
+	if (batadv_is_my_mac(unicast_packet->dest)) {
+

total: 0 errors, 0 warnings, 4 checks, 1281 lines checked

checkpatch next send.c:
>>>>>>>>
CHECK: Blank lines aren't necessary after an open brace '{'
#333: FILE: send.c:333:
+				  &bat_priv->forw_bcast_list, list) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#360: FILE: send.c:360:
+				  &bat_priv->forw_bat_list, list) {
+

total: 0 errors, 0 warnings, 2 checks, 382 lines checked

checkpatch next soft-interface.c:
>>>>>>>>
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
WARNING: Avoid CamelCase: <vhdr->h_vlan_TCI>
#170: FILE: soft-interface.c:170:
+		vid = ntohs(vhdr->h_vlan_TCI) & VLAN_VID_MASK;

total: 0 errors, 1 warnings, 0 checks, 662 lines checked

checkpatch next translation-table.c:
>>>>>>>>
CHECK: Blank lines aren't necessary before a close brace '}'
#116: FILE: translation-table.c:116:
+
+}

CHECK: Blank lines aren't necessary before a close brace '}'
#253: FILE: translation-table.c:253:
+
+}

CHECK: Blank lines aren't necessary before a close brace '}'
#631: FILE: translation-table.c:631:
+
+}

CHECK: Blank lines aren't necessary before a close brace '}'
#1053: FILE: translation-table.c:1053:
+
+}

CHECK: Blank lines aren't necessary before a close brace '}'
#1829: FILE: translation-table.c:1829:
+
+}

CHECK: Blank lines aren't necessary before a close brace '}'
#2356: FILE: translation-table.c:2356:
+
+}

CHECK: Blank lines aren't necessary before a close brace '}'
#2553: FILE: translation-table.c:2553:
+
+}

total: 0 errors, 0 warnings, 7 checks, 2579 lines checked

checkpatch next unicast.c:
>>>>>>>>
CHECK: Blank lines aren't necessary after an open brace '{'
#136: FILE: unicast.c:136:
+	list_for_each_entry(tfp, head, list) {
+

CHECK: Blank lines aren't necessary after an open brace '{'
#165: FILE: unicast.c:165:
+	if (!list_empty(head)) {
+

total: 0 errors, 0 warnings, 2 checks, 480 lines checked

checkpatch next vis.c:
>>>>>>>>
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3368.
Use of uninitialized value $ms_size in pattern match (m//) at /home/batman/build_test/linux-next/scripts/checkpatch.pl line 3371.



More information about the linux-merge mailing list