Project

General

Profile

Bug #12654

Updated by Toomas Soome about 2 months ago

Build errors on SPARC with NULL pointers:
<pre>
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_hw.c:220 nxge_intr() warn: variable dereferenced before check 'ldvp' (see line 208)

../../common/io/nxge/nxge_hio_guest.c: In function 'nxge_hio_vr_release':
../../common/io/nxge/nxge_hio_guest.c:575: error: assignment makes integer from pointer without a cast

/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_ipp.c:769 nxge_ipp_eccue_valid_check() warn: variable dereferenced before check 'valid' (see line 735)

/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_rxdma.c:3059 nxge_rx_err_evnts() warn: if statement not indented

/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/npi/npi_fflp.c:653 npi_fflp_fcram_entry_write() warn: inconsistent indenting
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/npi/npi_fflp.c:721 npi_fflp_fcram_entry_read() warn: inconsistent indenting
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/npi/npi_fflp.c:733 npi_fflp_fcram_entry_read() warn: inconsistent indenting
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/npi/npi_fflp.c:747 npi_fflp_fcram_entry_read() warn: inconsistent indenting
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/npi/npi_fflp.c:974 npi_fflp_cfg_fcram_partition() warn: inconsistent indenting
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/npi/npi_fflp.c:1575 npi_fflp_cfg_enet_vlan_table_assoc() warn: inconsistent indenting

/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_virtual.c:2852 nxge_set_hw_vlan_class_config() warn: always true condition '(vmap->map_to >= 0) => (0-255 >= 0)'
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_virtual.c:2914 nxge_set_hw_mac_class_config() warn: always true condition '(mac_map->map_to >= 0) => (0-255 >= 0)'

/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_send.c:1225 nxge_lso_info_get() warn: variable dereferenced before check 'mss' (see line 1222)

../../common/io/nxge/nxge_main.c: In function 'nxge_dma_mem_alloc':
../../common/io/nxge/nxge_main.c:3529: error: assignment makes integer from pointer without a cast
../../common/io/nxge/nxge_main.c:3536: error: comparison between pointer and integer
../../common/io/nxge/nxge_main.c: In function 'nxge_dma_mem_free':
../../common/io/nxge/nxge_main.c:3649: error: assignment makes integer from pointer without a cast
../../common/io/nxge/nxge_main.c: In function 'nxge_dma_free_rx_data_buf':
../../common/io/nxge/nxge_main.c:3710: error: assignment makes integer from pointer without a cast
../../common/io/nxge/nxge_main.c:3714: error: assignment makes integer from pointer without a cast

/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_mac.c:1540 nxge_n2_serdes_init() warn: if statement not indented
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_mac.c:2005 nxge_n2_kt_serdes_init() warn: inconsistent indenting
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_mac.c:2034 nxge_n2_kt_serdes_init() warn: inconsistent indenting
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_mac.c:2522 nxge_1G_serdes_init() warn: inconsistent indenting
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_mac.c:2536 nxge_1G_serdes_init() warn: inconsistent indenting
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_mac.c:4640 nxge_rx_mac_reset() warn: if statement not indented
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_mac.c:4644 nxge_rx_mac_reset() warn: if statement not indented
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_mac.c:7130 nxge_mac_intr() warn: curly braces intended?
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_mac.c:7133 nxge_mac_intr() warn: inconsistent indenting
/code/illumos-gate/usr/src/tools/proto/root_sparc-nd/opt/onbld/bin/sparc/smatch: ../../common/io/nxge/nxge_mac.c:7518 nxge_hswap_phy_present() warn: inconsistent indenting
</pre>

Testing done: build/install/boot

Back