[SCM] Samba Shared Repository - branch master updated

Andrew Bartlett abartlet at samba.org
Thu Mar 1 08:53:03 UTC 2018


The branch, master has been updated
       via  1aabfff libsmb: Remove incorrect fall through comment in trusts_util.c
       via  d4ccca2 s3:spoolss: Remove incorrect fall through comment in srv_spoolss_nt.c
       via  a31147d s4:lib: Add FALL_THROUGH statements in http.c
       via  1563f39 s4:dsdb: Add FALL_THROUGH statements in password_hash.c
       via  7619442 s3:winbindd: Add FALL_THROUGH statements in idmap_autorid.c
       via  5ba0b72 s3:utils: Add FALL_THROUGH statements in ntlm_auth.c
       via  4515312 s3:utils: Add FALL_THROUGH statements in net_registry_check.c
       via  29b5de2 s3:smbd: Add FALL_THROUGH statements in reply.c
       via  3e2e409 s3:rpcclient: Add FALL_THROUGH statements in rpcclient.c
       via  cafe8ac s3:lsa: Add FALL_THROUGH statements in srv_lsa_nt.c
       via  824fa2c s3:modules: Add FALL_THROUGH statements in getdate.c
       via  f5d69d7 s3:libnet: Add FALL_THROUGH statements in libnet_join.c
       via  00defe7 nsswitch: Add FALL_THROUGH statements in pam_winbind.c
       via  2a646a7 auth:gensec: Add FALL_THROUGH statements in spnego.c
       via  d123638 auth:credentials: Add FALL_THROUGH statements in credentials_secrets.c
       via  856cda1 s4:torture: Add FALL_THROUGH statements in rpc/spoolss.c
       via  50d3a9e s4:torture: Add FALL_THROUGH statements in basic/misc.c
       via  1508ff9 s4:rpc_server: Add FALL_THROUGH statements in dcesrv_srvsvc.c
       via  e2f1255 s4:auth: Add FALL_THROUGH statements in gensec_krb5.c
       via  69fa5ec s4:auth: Add FALL_THROUGH statements in auth_sam.c
       via  ef3ac40 s4:auth: Add FALL_THROUGH statements in auth_util.c
       via  487bd0a s4:samdb: Add FALL_THROUGH statements in linked_attributes.c
       via  94d69cd s4:samdb: Add FALL_THROUGH statements in cracknames.c
       via  d17cb7c s3:rpc_server: Add FALL_THROUGH statements in rpc_server.c
       via  7b946e3 s3:utils: Add FALL_THROUGH statements in net_rpc_conf.c
       via  3631576 s3:utils: Add FALL_THROUGH statements in net_conf.c
       via  273ef59 s3:utils: Add FALL_THROUGH statements in regedit.c
       via  964b2d4 s3:smbd: Add FALL_THROUGH statements in trans2.c
       via  a4623fa s3:smbd: Add FALL_THROUGH statements in nttrans.c
       via  4744b77 s3:modules: Add FALL_THROUGH statements in vfs_acl_common.c
       via  2cc1306 s3:libsmb: Add FALL_THROUGH statements in dsgetdcname.c
       via  3b9aa1c s3:lib: Add FALL_THROUGH statements in util_sd.c
       via  feeb49f s3:lib: Add FALL_THROUGH statements in sysacls.c
       via  3f1a4b7 s3:lib: Add FALL_THROUGH statements in cbuf.c
       via  03ed979 s3:auth: Add FALL_THROUGH statements in pampass.c
       via  6744e8c s3:auth: Add FALL_THROUGH statements in auth_sam.c
       via  64c9ca3 librpc:ndr: Add FALL_THROUGH statements in ndr_cab.c
       via  7217219 libgpo: Add FALL_THROUGH statements in gpo_sec.c
       via  1741072 lib:tdb: Add FALL_THROUGH statements in common/summary.c
       via  bdd1de1 lib:tdb: Add FALL_THROUGH statements in tdbtool.c
       via  20e3a93 lib:tdb: Add FALL_THROUGH statements in hash.c
       via  16d14fa s3:lib: Add FALL_THROUGH statements in util_str.c
       via  9fed456 s3:lib: Add FALL_THROUGH statements in util_path.c
       via  d78d1a7 s3:lib: Add FALL_THROUGH statements in substitute_generic.c
       via  a3fffd9 lib:util: Add FALL_THROUGH statements in util_file.c
       via  8da568e lib:util: Add FALL_THROUGH statements in charset/charset_macosxfs.c
       via  37c6377 lib:util: Add FALL_THROUGH statements in substitute.c
       via  2683140 lib:param: Add FALL_THROUGH statements in loadparm.c
       via  7adf497 lib:ldb: Add FALL_THROUGH statements in ldb_map/ldb_map_outbound.c
       via  9d2296e lib:ldb: Add FALL_THROUGH statements in ldb_map/ldb_map.c
       via  3429e9d lib:ldb: Add FALL_THROUGH statements in ldb_map/ldb_map_inbound.c
       via  2b01bd0 lib:ldb: Add FALL_THROUGH statements in common/ldb_dn.c
       via  a5104e6 lib:replace: Add FALL_THROUGH statements in strptime.c
       via  05dae36 lib:replace: Add FALL_THROUGH support
       via  76535df lib:texpect: Avoid some compiler warnings
       via  d31a8cd s4:lib:com: Fix function declartions
      from  c21d4bc py3compat: add strings describing bytes/unicode in both versions

https://git.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 1aabfff8a6ecd57ac6475667c2103eb8e3ec6bf3
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:06:48 2017 +0200

    libsmb: Remove incorrect fall through comment in trusts_util.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>
    
    Autobuild-User(master): Andrew Bartlett <abartlet at samba.org>
    Autobuild-Date(master): Thu Mar  1 09:52:37 CET 2018 on sn-devel-144

commit d4ccca2d66536a24afe702dbb969a6ab878d914d
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:11:24 2017 +0200

    s3:spoolss: Remove incorrect fall through comment in srv_spoolss_nt.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit a31147d675c6d557d2b139263134f8617108ba6c
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:27:38 2017 +0200

    s4:lib: Add FALL_THROUGH statements in http.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 1563f392c838798824bec248221a56125350e45c
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:26:38 2017 +0200

    s4:dsdb: Add FALL_THROUGH statements in password_hash.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 7619442a79ca0cff0d2edcab7a9bea36913683bf
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:25:27 2017 +0200

    s3:winbindd: Add FALL_THROUGH statements in idmap_autorid.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 5ba0b72fa3c7d131833e9055a00be2530e73ce75
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:24:23 2017 +0200

    s3:utils: Add FALL_THROUGH statements in ntlm_auth.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 45153120cb03c7e77a7b88f10679ebc7218519c9
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:18:09 2017 +0200

    s3:utils: Add FALL_THROUGH statements in net_registry_check.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 29b5de26df8e562387540ad89b92fc94912ed8a7
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:15:31 2017 +0200

    s3:smbd: Add FALL_THROUGH statements in reply.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 3e2e4096ccb9095847874cbbd89be054299d19ae
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:13:42 2017 +0200

    s3:rpcclient: Add FALL_THROUGH statements in rpcclient.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit cafe8ac9653a8ca9adbbc72e29fd71bf8c13a2d5
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:12:43 2017 +0200

    s3:lsa: Add FALL_THROUGH statements in srv_lsa_nt.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 824fa2c4dbba252cfa778027f0ea7fdfa4fda894
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 13 11:08:36 2017 +0200

    s3:modules: Add FALL_THROUGH statements in getdate.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit f5d69d7134ef3a9b73a9d2dcecac42e69319548c
Author: Andreas Schneider <asn at samba.org>
Date:   Thu Jul 27 16:59:40 2017 +0200

    s3:libnet: Add FALL_THROUGH statements in libnet_join.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 00defe71009246703a23edd3cf20292821569a86
Author: Andreas Schneider <asn at samba.org>
Date:   Thu Jul 27 16:56:27 2017 +0200

    nsswitch: Add FALL_THROUGH statements in pam_winbind.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 2a646a7485eb2e042e7d301b50e418a1aca09b90
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 18:24:26 2017 +0200

    auth:gensec: Add FALL_THROUGH statements in spnego.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit d123638a82e14d457572f2d97c0283796b6a1973
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 18:23:31 2017 +0200

    auth:credentials: Add FALL_THROUGH statements in credentials_secrets.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 856cda10375e2891d419d5c471df494ddc94976a
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:41:26 2017 +0200

    s4:torture: Add FALL_THROUGH statements in rpc/spoolss.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 50d3a9e70d2c5e86d3d26aeb369ebd7a83b60375
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:39:52 2017 +0200

    s4:torture: Add FALL_THROUGH statements in basic/misc.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 1508ff9cc45ea7e848e3373f66c83006a5a47d37
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:39:04 2017 +0200

    s4:rpc_server: Add FALL_THROUGH statements in dcesrv_srvsvc.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit e2f1255840b0066c223f1f9caf7a3865fa4021a3
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:33:12 2017 +0200

    s4:auth: Add FALL_THROUGH statements in gensec_krb5.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 69fa5ecb6acaacdc0fbd8942baa3bbba1dabb75a
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:30:53 2017 +0200

    s4:auth: Add FALL_THROUGH statements in auth_sam.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit ef3ac405bf436fa6fd7daf20e0c90856dae8237f
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:29:55 2017 +0200

    s4:auth: Add FALL_THROUGH statements in auth_util.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 487bd0a1e94c121daaf98acbb575612db297a520
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:28:11 2017 +0200

    s4:samdb: Add FALL_THROUGH statements in linked_attributes.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 94d69cd4b8d9c50908615e152766af3c93011e4a
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:21:02 2017 +0200

    s4:samdb: Add FALL_THROUGH statements in cracknames.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit d17cb7cebfa7d4525dca24668a35bd1d26034c42
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:42:46 2017 +0200

    s3:rpc_server: Add FALL_THROUGH statements in rpc_server.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 7b946e3480fc964895e1ef9bac340361aa209aa1
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:58:40 2017 +0200

    s3:utils: Add FALL_THROUGH statements in net_rpc_conf.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 36315769b87d8de0482cb5090a9ddcda975f3c45
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:57:10 2017 +0200

    s3:utils: Add FALL_THROUGH statements in net_conf.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 273ef597616dc41620860781fe0e19cc747fe931
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:55:35 2017 +0200

    s3:utils: Add FALL_THROUGH statements in regedit.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 964b2d4aae5a4700a0e74725e6246f856df248b7
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:52:06 2017 +0200

    s3:smbd: Add FALL_THROUGH statements in trans2.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit a4623fac64b7471e3dbde5cfca8f9dfa0ec28761
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:50:18 2017 +0200

    s3:smbd: Add FALL_THROUGH statements in nttrans.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 4744b777f5143b8956afb29ea2b35a8248c7daf9
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:53:45 2017 +0200

    s3:modules: Add FALL_THROUGH statements in vfs_acl_common.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 2cc130683e9b885739b4ae7952f1618277100bc2
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:36:50 2017 +0200

    s3:libsmb: Add FALL_THROUGH statements in dsgetdcname.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 3b9aa1c625b0cc5ef55cc74602b6160443691e46
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:54:48 2017 +0200

    s3:lib: Add FALL_THROUGH statements in util_sd.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit feeb49f4cfe6260daba1c873d27cd848f063e8bd
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:51:08 2017 +0200

    s3:lib: Add FALL_THROUGH statements in sysacls.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 3f1a4b749d4157d075cd9146ab5d10d5d0e6ceec
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:25:30 2017 +0200

    s3:lib: Add FALL_THROUGH statements in cbuf.c

commit 03ed979eb0cfa37fb25cbaacc9e7a6fd871a8ec9
Author: Andreas Schneider <asn at samba.org>
Date:   Thu Jul 27 16:55:23 2017 +0200

    s3:auth: Add FALL_THROUGH statements in pampass.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 6744e8c7d4fb9fcb27cf940608118bd28ea2af2e
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:35:28 2017 +0200

    s3:auth: Add FALL_THROUGH statements in auth_sam.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 64c9ca3a3df42dd19e0ed2981d2e1b3cb0990ca4
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:03:09 2017 +0200

    librpc:ndr: Add FALL_THROUGH statements in ndr_cab.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 721721911595c649b5cb549f19873eb8f07d2590
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:43:53 2017 +0200

    libgpo: Add FALL_THROUGH statements in gpo_sec.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 17410725065da5040f9c6eae2383a960b7712156
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 18:28:12 2017 +0200

    lib:tdb: Add FALL_THROUGH statements in common/summary.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit bdd1de17109f9f4ed6435851d74b97d19ffe6da1
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 16:58:00 2017 +0200

    lib:tdb: Add FALL_THROUGH statements in tdbtool.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 20e3a93c3b5b580abef4b2ab7eccc5f1be5d4b77
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 16:55:10 2017 +0200

    lib:tdb: Add FALL_THROUGH statements in hash.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 16d14fa86321f30ce2798aa5e4b9be493c267807
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:25:20 2017 +0200

    s3:lib: Add FALL_THROUGH statements in util_str.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 9fed4562cae3e7e2b528b94176b6063522f332eb
Author: Andreas Schneider <asn at samba.org>
Date:   Thu Jul 27 16:57:38 2017 +0200

    s3:lib: Add FALL_THROUGH statements in util_path.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit d78d1a71ffa166a95729e273a68e012f2965120e
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 17:22:44 2017 +0200

    s3:lib: Add FALL_THROUGH statements in substitute_generic.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit a3fffd970c38e288476643fbea6cfa1943213b8d
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 18:41:25 2017 +0200

    lib:util: Add FALL_THROUGH statements in util_file.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 8da568efea444af26c7f63ba73de65c4a9f24e7d
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 18:40:14 2017 +0200

    lib:util: Add FALL_THROUGH statements in charset/charset_macosxfs.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 37c637766eabd1123b259acbb3ae4eeb6b10de4a
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 16:49:50 2017 +0200

    lib:util: Add FALL_THROUGH statements in substitute.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 2683140674fe37f449029870f0a2b6588b10a2b0
Author: Andreas Schneider <asn at samba.org>
Date:   Thu Jul 27 15:20:57 2017 +0200

    lib:param: Add FALL_THROUGH statements in loadparm.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 7adf497443f40aa381c242e5d72970369b021958
Author: Andreas Schneider <asn at samba.org>
Date:   Thu Jul 27 15:20:04 2017 +0200

    lib:ldb: Add FALL_THROUGH statements in ldb_map/ldb_map_outbound.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 9d2296e070157ba9d4d4f80714ba8b1ad6691117
Author: Andreas Schneider <asn at samba.org>
Date:   Thu Jul 27 15:19:23 2017 +0200

    lib:ldb: Add FALL_THROUGH statements in ldb_map/ldb_map.c

commit 3429e9d9e59706ff2d17989530b67c74f80b879e
Author: Andreas Schneider <asn at samba.org>
Date:   Thu Jul 27 15:18:28 2017 +0200

    lib:ldb: Add FALL_THROUGH statements in ldb_map/ldb_map_inbound.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 2b01bd0475d6cb654f1ffbbc56152bf04d1a793d
Author: Andreas Schneider <asn at samba.org>
Date:   Thu Jul 27 15:17:21 2017 +0200

    lib:ldb: Add FALL_THROUGH statements in common/ldb_dn.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit a5104e6a396ded1c2f603eff5bf8fc89bfe3bb61
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 18:25:46 2017 +0200

    lib:replace: Add FALL_THROUGH statements in strptime.c
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 05dae361b388e71701a7c646f2787315e9bd5dfb
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 16:33:10 2017 +0200

    lib:replace: Add FALL_THROUGH support
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit 76535df3248b7f4e489d1555f27cedd151cd1a50
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 16:51:39 2017 +0200

    lib:texpect: Avoid some compiler warnings
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

commit d31a8cd800db3862cf7e6935f135705af647b1d0
Author: Andreas Schneider <asn at samba.org>
Date:   Wed Jul 26 18:16:43 2017 +0200

    s4:lib:com: Fix function declartions
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Andrew Bartlett <abartlet at samba.org>

-----------------------------------------------------------------------

Summary of changes:
 .ycm_extra_conf.py                                 |  1 +
 auth/credentials/credentials_secrets.c             |  3 +-
 auth/gensec/spnego.c                               |  2 +-
 lib/ldb/common/ldb_dn.c                            |  3 +-
 lib/ldb/ldb_map/ldb_map.c                          |  6 ++-
 lib/ldb/ldb_map/ldb_map_inbound.c                  |  6 ++-
 lib/ldb/ldb_map/ldb_map_outbound.c                 |  3 +-
 lib/param/loadparm.c                               |  3 +-
 lib/replace/replace.h                              |  9 ++++
 lib/replace/strptime.c                             |  6 ++-
 lib/replace/wscript                                | 36 ++++++++++++++++
 lib/tdb/common/hash.c                              | 50 +++++++++++-----------
 lib/tdb/common/summary.c                           |  3 +-
 lib/tdb/tools/tdbtool.c                            |  3 ++
 lib/texpect/texpect.c                              |  9 ++++
 lib/util/charset/charset_macosxfs.c                | 14 +++---
 lib/util/substitute.c                              |  2 +
 lib/util/util_file.c                               |  3 +-
 libgpo/gpo_sec.c                                   |  6 ++-
 librpc/idl/dcom.idl                                |  2 +-
 librpc/ndr/ndr_cab.c                               |  3 ++
 nsswitch/pam_winbind.c                             |  3 +-
 source3/auth/auth_sam.c                            |  4 ++
 source3/auth/pampass.c                             |  4 +-
 source3/lib/cbuf.c                                 |  3 +-
 source3/lib/substitute_generic.c                   |  2 +
 source3/lib/sysacls.c                              |  6 ++-
 source3/lib/util_path.c                            | 12 ++++--
 source3/lib/util_sd.c                              |  2 +
 source3/lib/util_str.c                             |  4 ++
 source3/libnet/libnet_join.c                       |  3 +-
 source3/libsmb/dsgetdcname.c                       |  2 +
 source3/libsmb/trusts_util.c                       |  1 -
 source3/modules/getdate.c                          |  5 ++-
 source3/modules/vfs_acl_common.c                   |  1 +
 source3/rpc_server/lsa/srv_lsa_nt.c                |  6 ++-
 source3/rpc_server/rpc_server.c                    |  3 +-
 source3/rpc_server/spoolss/srv_spoolss_nt.c        |  1 -
 source3/rpcclient/rpcclient.c                      |  2 +-
 source3/smbd/nttrans.c                             |  1 +
 source3/smbd/reply.c                               |  8 ++--
 source3/smbd/trans2.c                              |  3 +-
 source3/utils/net_conf.c                           |  8 ++++
 source3/utils/net_registry_check.c                 |  3 +-
 source3/utils/net_rpc_conf.c                       |  8 ++++
 source3/utils/ntlm_auth.c                          |  4 +-
 source3/utils/regedit.c                            |  3 +-
 source3/winbindd/idmap_autorid.c                   |  2 +-
 source4/auth/gensec/gensec_krb5.c                  |  5 +--
 source4/auth/ntlm/auth_sam.c                       |  2 +-
 source4/auth/ntlm/auth_util.c                      |  9 ++--
 source4/dsdb/samdb/cracknames.c                    |  2 +-
 source4/dsdb/samdb/ldb_modules/linked_attributes.c |  3 +-
 source4/dsdb/samdb/ldb_modules/password_hash.c     |  2 +-
 source4/lib/com/classes/simple.c                   |  4 +-
 source4/lib/com/main.c                             |  4 +-
 source4/lib/http/http.c                            |  2 +-
 source4/rpc_server/srvsvc/dcesrv_srvsvc.c          |  4 ++
 source4/torture/basic/misc.c                       |  2 +
 source4/torture/rpc/spoolss.c                      |  2 +
 60 files changed, 225 insertions(+), 93 deletions(-)


Changeset truncated at 500 lines:

diff --git a/.ycm_extra_conf.py b/.ycm_extra_conf.py
index c96b59e..97122a9 100644
--- a/.ycm_extra_conf.py
+++ b/.ycm_extra_conf.py
@@ -47,6 +47,7 @@ flags = [
     '-D_XOPEN_SOURCE_EXTENDED=1',
     '-DAD_DC_BUILD_IS_ENABLED=1',
     '-DHAVE_IPV6=1',
+    '-DFALL_THROUGH',
     '-I/usr/local/include',
     '-I.',
     '-Iauth',
diff --git a/auth/credentials/credentials_secrets.c b/auth/credentials/credentials_secrets.c
index beb2928..ae1d23b 100644
--- a/auth/credentials/credentials_secrets.c
+++ b/auth/credentials/credentials_secrets.c
@@ -372,7 +372,8 @@ _PUBLIC_ NTSTATUS cli_credentials_set_machine_account_db_ctx(struct cli_credenti
 				if (security != SEC_ADS) {
 					break;
 				}
-				/* fall through */
+
+				FALL_THROUGH;
 			case ROLE_ACTIVE_DIRECTORY_DC:
 				use_kerberos = CRED_AUTO_USE_KERBEROS;
 				break;
diff --git a/auth/gensec/spnego.c b/auth/gensec/spnego.c
index 56f9be4..0b3fbdc 100644
--- a/auth/gensec/spnego.c
+++ b/auth/gensec/spnego.c
@@ -1649,7 +1649,7 @@ static struct tevent_req *gensec_spnego_update_send(TALLOC_CTX *mem_ctx,
 			return tevent_req_post(req, ev);
 		}
 
-		/* fall through */
+		FALL_THROUGH;
 	case SPNEGO_CLIENT_START:
 	case SPNEGO_SERVER_START:
 
diff --git a/lib/ldb/common/ldb_dn.c b/lib/ldb/common/ldb_dn.c
index b23ee17..dfeb600 100644
--- a/lib/ldb/common/ldb_dn.c
+++ b/lib/ldb/common/ldb_dn.c
@@ -629,7 +629,8 @@ static bool ldb_dn_explode(struct ldb_dn *dn)
 					l++;
 					break;
 				}
-				/* fall through */
+
+				FALL_THROUGH;
 			case '\"':
 			case '<':
 			case '>':
diff --git a/lib/ldb/ldb_map/ldb_map.c b/lib/ldb/ldb_map/ldb_map.c
index f2a86fe..b453dff 100644
--- a/lib/ldb/ldb_map/ldb_map.c
+++ b/lib/ldb/ldb_map/ldb_map.c
@@ -523,7 +523,8 @@ struct ldb_dn *ldb_dn_map_local(struct ldb_module *module, void *mem_ctx, struct
 					  "used in DN!", ldb_dn_get_component_name(dn, i));
 				goto failed;
 			}
-			/* fall through */
+
+			FALL_THROUGH;
 		case LDB_MAP_KEEP:
 		case LDB_MAP_RENAME:
 		case LDB_MAP_RENDROP:
@@ -599,7 +600,8 @@ struct ldb_dn *ldb_dn_map_remote(struct ldb_module *module, void *mem_ctx, struc
 					  "used in DN!", ldb_dn_get_component_name(dn, i));
 				goto failed;
 			}
-			/* fall through */
+
+			FALL_THROUGH;
 		case LDB_MAP_KEEP:
 		case LDB_MAP_RENAME:
 		case LDB_MAP_RENDROP:
diff --git a/lib/ldb/ldb_map/ldb_map_inbound.c b/lib/ldb/ldb_map/ldb_map_inbound.c
index 461e681..861c4c1 100644
--- a/lib/ldb/ldb_map/ldb_map_inbound.c
+++ b/lib/ldb/ldb_map/ldb_map_inbound.c
@@ -87,7 +87,8 @@ static int ldb_msg_el_partition(struct ldb_module *module, enum ldb_request_type
 			el = ldb_msg_el_map_local(module, remote, map, old);
 			break;
 		}
-		/* fall through */
+
+		FALL_THROUGH;
 	case LDB_MAP_IGNORE:
 		goto local;
 
@@ -99,7 +100,8 @@ static int ldb_msg_el_partition(struct ldb_module *module, enum ldb_request_type
 				  map->local_name);
 			goto local;
 		}
-		/* fall through */
+
+		FALL_THROUGH;
 	case LDB_MAP_KEEP:
 	case LDB_MAP_RENAME:
 		el = ldb_msg_el_map_local(module, remote, map, old);
diff --git a/lib/ldb/ldb_map/ldb_map_outbound.c b/lib/ldb/ldb_map/ldb_map_outbound.c
index fd25c36..1f1a7e8 100644
--- a/lib/ldb/ldb_map/ldb_map_outbound.c
+++ b/lib/ldb/ldb_map/ldb_map_outbound.c
@@ -330,7 +330,8 @@ static int ldb_msg_el_merge(struct ldb_module *module, struct ldb_message *local
 				  attr_name);
 			return LDB_SUCCESS;
 		}
-		/* fall through */
+
+		FALL_THROUGH;
 	case LDB_MAP_KEEP:
 	case LDB_MAP_RENAME:
 	case LDB_MAP_RENDROP:
diff --git a/lib/param/loadparm.c b/lib/param/loadparm.c
index efad4a1..b46700d 100644
--- a/lib/param/loadparm.c
+++ b/lib/param/loadparm.c
@@ -2058,7 +2058,8 @@ void lpcfg_print_parameter(struct parm_struct *p, void *ptr, FILE * f)
 
 		case P_CMDLIST:
 			list_sep = " ";
-			/* fall through */
+
+			FALL_THROUGH;
 		case P_LIST:
 			if ((char ***)ptr && *(char ***)ptr) {
 				char **list = *(char ***)ptr;
diff --git a/lib/replace/replace.h b/lib/replace/replace.h
index 128978c..e2a5541 100644
--- a/lib/replace/replace.h
+++ b/lib/replace/replace.h
@@ -922,6 +922,15 @@ void rep_setproctitle(const char *fmt, ...) PRINTF_ATTRIBUTE(1, 2);
 #define setproctitle_init rep_setproctitle_init
 void rep_setproctitle_init(int argc, char *argv[], char *envp[]);
 #endif
+
+#ifndef FALL_THROUGH
+# ifdef HAVE_FALLTHROUGH_ATTRIBUTE
+#  define FALL_THROUGH __attribute__ ((fallthrough))
+# else /* HAVE_FALLTHROUGH_ATTRIBUTE */
+#  define FALL_THROUGH ((void)0)
+# endif /* HAVE_FALLTHROUGH_ATTRIBUTE */
+#endif /* FALL_THROUGH */
+
 bool nss_wrapper_enabled(void);
 bool nss_wrapper_hosts_enabled(void);
 bool socket_wrapper_enabled(void);
diff --git a/lib/replace/strptime.c b/lib/replace/strptime.c
index 20e5d8c..bbc7422 100644
--- a/lib/replace/strptime.c
+++ b/lib/replace/strptime.c
@@ -462,7 +462,8 @@ strptime_internal (rp, fmt, tm, decided, era_cnt)
 	      *decided = raw;
 	    }
 #endif
-	  /* Fall through.  */
+
+	  FALL_THROUGH;
 	case 'D':
 	  /* Match standard day format.  */
 	  if (!recursive (HERE_D_FMT))
@@ -611,7 +612,8 @@ strptime_internal (rp, fmt, tm, decided, era_cnt)
 	      *decided = raw;
 	    }
 #endif
-	  /* Fall through.  */
+
+	  FALL_THROUGH;
 	case 'T':
 	  if (!recursive (HERE_T_FMT))
 	    return NULL;
diff --git a/lib/replace/wscript b/lib/replace/wscript
index a2e2d11..534062e 100644
--- a/lib/replace/wscript
+++ b/lib/replace/wscript
@@ -249,6 +249,42 @@ def configure(conf):
                     headers='stdint.h sys/atomic.h',
                     msg='Checking for atomic_add_32 compiler builtin')
 
+    conf.CHECK_CODE('''
+                    #define FALL_THROUGH __attribute__((fallthrough))
+
+                    enum direction_e {
+                        UP = 0,
+                        DOWN,
+                    };
+
+                    int main(void) {
+                        enum direction_e key = UP;
+                        int i = 10;
+                        int j = 0;
+
+                        switch (key) {
+                        case UP:
+                            i = 5;
+                            FALL_THROUGH;
+                        case DOWN:
+                            j = i * 2;
+                            break;
+                        default:
+                            break;
+                        }
+
+                        if (j < i) {
+                            return 1;
+                        }
+
+                        return 0;
+                    }
+                    ''',
+                    'HAVE_FALLTHROUGH_ATTRIBUTE',
+                    addmain=False,
+                    cflags='-Werror',
+                    msg='Checking for fallthrough attribute')
+
     # these may be builtins, so we need the link=False strategy
     conf.CHECK_FUNCS('strdup memmem printf memset memcpy memmove strcpy strncpy bzero', link=False)
 
diff --git a/lib/tdb/common/hash.c b/lib/tdb/common/hash.c
index 1eed722..4de7ba9 100644
--- a/lib/tdb/common/hash.c
+++ b/lib/tdb/common/hash.c
@@ -232,16 +232,16 @@ static uint32_t hashlittle( const void *key, size_t length )
     switch(length)
     {
     case 12: c+=k[2]; b+=k[1]; a+=k[0]; break;
-    case 11: c+=((uint32_t)k8[10])<<16;  /* fall through */
-    case 10: c+=((uint32_t)k8[9])<<8;    /* fall through */
-    case 9 : c+=k8[8];                   /* fall through */
+    case 11: c+=((uint32_t)k8[10])<<16; FALL_THROUGH;
+    case 10: c+=((uint32_t)k8[9])<<8;   FALL_THROUGH;
+    case 9 : c+=k8[8];                  FALL_THROUGH;
     case 8 : b+=k[1]; a+=k[0]; break;
-    case 7 : b+=((uint32_t)k8[6])<<16;   /* fall through */
-    case 6 : b+=((uint32_t)k8[5])<<8;    /* fall through */
-    case 5 : b+=k8[4];                   /* fall through */
+    case 7 : b+=((uint32_t)k8[6])<<16;  FALL_THROUGH;
+    case 6 : b+=((uint32_t)k8[5])<<8;   FALL_THROUGH;
+    case 5 : b+=k8[4];                  FALL_THROUGH;
     case 4 : a+=k[0]; break;
-    case 3 : a+=((uint32_t)k8[2])<<16;   /* fall through */
-    case 2 : a+=((uint32_t)k8[1])<<8;    /* fall through */
+    case 3 : a+=((uint32_t)k8[2])<<16;  FALL_THROUGH;
+    case 2 : a+=((uint32_t)k8[1])<<8;   FALL_THROUGH;
     case 1 : a+=k8[0]; break;
     case 0 : return c;
     }
@@ -268,23 +268,23 @@ static uint32_t hashlittle( const void *key, size_t length )
              b+=k[2]+(((uint32_t)k[3])<<16);
              a+=k[0]+(((uint32_t)k[1])<<16);
              break;
-    case 11: c+=((uint32_t)k8[10])<<16;     /* fall through */
+    case 11: c+=((uint32_t)k8[10])<<16;      FALL_THROUGH;
     case 10: c+=k[4];
              b+=k[2]+(((uint32_t)k[3])<<16);
              a+=k[0]+(((uint32_t)k[1])<<16);
              break;
-    case 9 : c+=k8[8];                      /* fall through */
+    case 9 : c+=k8[8];                       FALL_THROUGH;
     case 8 : b+=k[2]+(((uint32_t)k[3])<<16);
              a+=k[0]+(((uint32_t)k[1])<<16);
              break;
-    case 7 : b+=((uint32_t)k8[6])<<16;      /* fall through */
+    case 7 : b+=((uint32_t)k8[6])<<16;       FALL_THROUGH;
     case 6 : b+=k[2];
              a+=k[0]+(((uint32_t)k[1])<<16);
              break;
-    case 5 : b+=k8[4];                      /* fall through */
+    case 5 : b+=k8[4];                       FALL_THROUGH;
     case 4 : a+=k[0]+(((uint32_t)k[1])<<16);
              break;
-    case 3 : a+=((uint32_t)k8[2])<<16;      /* fall through */
+    case 3 : a+=((uint32_t)k8[2])<<16;       FALL_THROUGH;
     case 2 : a+=k[0];
              break;
     case 1 : a+=k8[0];
@@ -316,19 +316,19 @@ static uint32_t hashlittle( const void *key, size_t length )
     }
 
     /*-------------------------------- last block: affect all 32 bits of (c) */
-    switch(length)                   /* all the case statements fall through */
+    switch(length)
     {
-    case 12: c+=((uint32_t)k[11])<<24;
-    case 11: c+=((uint32_t)k[10])<<16;
-    case 10: c+=((uint32_t)k[9])<<8;
-    case 9 : c+=k[8];
-    case 8 : b+=((uint32_t)k[7])<<24;
-    case 7 : b+=((uint32_t)k[6])<<16;
-    case 6 : b+=((uint32_t)k[5])<<8;
-    case 5 : b+=k[4];
-    case 4 : a+=((uint32_t)k[3])<<24;
-    case 3 : a+=((uint32_t)k[2])<<16;
-    case 2 : a+=((uint32_t)k[1])<<8;
+    case 12: c+=((uint32_t)k[11])<<24; FALL_THROUGH;
+    case 11: c+=((uint32_t)k[10])<<16; FALL_THROUGH;
+    case 10: c+=((uint32_t)k[9])<<8;   FALL_THROUGH;
+    case 9 : c+=k[8];                  FALL_THROUGH;
+    case 8 : b+=((uint32_t)k[7])<<24;  FALL_THROUGH;
+    case 7 : b+=((uint32_t)k[6])<<16;  FALL_THROUGH;
+    case 6 : b+=((uint32_t)k[5])<<8;   FALL_THROUGH;
+    case 5 : b+=k[4];                  FALL_THROUGH;
+    case 4 : a+=((uint32_t)k[3])<<24;  FALL_THROUGH;
+    case 3 : a+=((uint32_t)k[2])<<16;  FALL_THROUGH;
+    case 2 : a+=((uint32_t)k[1])<<8;   FALL_THROUGH;
     case 1 : a+=k[0];
              break;
     case 0 : return c;
diff --git a/lib/tdb/common/summary.c b/lib/tdb/common/summary.c
index d786132..c9b5bc4 100644
--- a/lib/tdb/common/summary.c
+++ b/lib/tdb/common/summary.c
@@ -151,7 +151,8 @@ _PUBLIC_ char *tdb_summary(struct tdb_context *tdb)
 				rec.rec_len = tdb_dead_space(tdb, off)
 					- sizeof(rec);
 			}
-			/* Fall through */
+
+			FALL_THROUGH;
 		case TDB_DEAD_MAGIC:
 			tally_add(&dead, rec.rec_len);
 			break;
diff --git a/lib/tdb/tools/tdbtool.c b/lib/tdb/tools/tdbtool.c
index e3535b9..d8bacdb 100644
--- a/lib/tdb/tools/tdbtool.c
+++ b/lib/tdb/tools/tdbtool.c
@@ -930,10 +930,13 @@ int main(int argc, char *argv[])
 		break;
 	case 5:
 		arg2 = tdb_convert_string(argv[4],&arg2len);
+		FALL_THROUGH;
 	case 4:
 		arg1 = tdb_convert_string(argv[3],&arg1len);
+		FALL_THROUGH;
 	case 3:
 		cmdname = argv[2];
+		FALL_THROUGH;
 	default:
 		do_command();
 		break;
diff --git a/lib/texpect/texpect.c b/lib/texpect/texpect.c
index b553de8..dd78670 100644
--- a/lib/texpect/texpect.c
+++ b/lib/texpect/texpect.c
@@ -434,6 +434,9 @@ int main(int argc, const char **argv)
 	switch (pid) {
 		case -1:
 			err(1, "Failed to fork");
+
+			/* Never reached */
+			return 1;
 		case 0:
 
 			if(setsid()<0)
@@ -448,6 +451,9 @@ int main(int argc, const char **argv)
 			/* texpect <expect_instructions> <progname> [<args>] */
 			execvp(program, program_args);
 			err(1, "Failed to exec: %s", program);
+
+			/* Never reached */
+			return 1;
 		default:
 			close(slave);
 			{
@@ -462,4 +468,7 @@ int main(int argc, const char **argv)
 
 			return eval_parent(pid);
 	}
+
+	/* Never reached */
+	return 1;
 }
diff --git a/lib/util/charset/charset_macosxfs.c b/lib/util/charset/charset_macosxfs.c
index 895277d..24e21fd 100644
--- a/lib/util/charset/charset_macosxfs.c
+++ b/lib/util/charset/charset_macosxfs.c
@@ -457,10 +457,11 @@ static size_t macosxfs_encoding_pull(
 
 	switch(result) {
 	case kCFStringEncodingConversionSuccess:
-		if (*inbytesleft == srcCharsUsed)
+		if (*inbytesleft == srcCharsUsed) {
 			break;
-		else
-			; /*fall through*/
+		}
+
+		FALL_THROUGH;
 	case kCFStringEncodingInsufficientOutputBufferLength:
 		debug_out("String conversion: "
 			  "Output buffer too small\n");
@@ -546,10 +547,11 @@ static size_t macosxfs_encoding_push(
 
 	switch(result) {
 	case kCFStringEncodingConversionSuccess:
-		if (*inbytesleft/2 == srcCharsUsed)
+		if (*inbytesleft/2 == srcCharsUsed) {
 			break;
-		else
-			; /*fall through*/
+		}
+
+		FALL_THROUGH;
 	case kCFStringEncodingInsufficientOutputBufferLength:
 		debug_out("String conversion: "
 			  "Output buffer too small\n");
diff --git a/lib/util/substitute.c b/lib/util/substitute.c
index 49adeaf..2c18257 100644
--- a/lib/util/substitute.c
+++ b/lib/util/substitute.c
@@ -84,6 +84,7 @@ static void string_sub2(char *s,const char *pattern, const char *insert, size_t
 					p[i] = insert[i];
 					break;
 				}
+				FALL_THROUGH;
 			case '`':
 			case '"':
 			case '\'':
@@ -98,6 +99,7 @@ static void string_sub2(char *s,const char *pattern, const char *insert, size_t
 					 * not replacing unsafe chars */
 					break;
 				}
+				FALL_THROUGH;
 			default:
 				p[i] = insert[i];
 			}
diff --git a/lib/util/util_file.c b/lib/util/util_file.c
index ac82060..499e8c4 100644
--- a/lib/util/util_file.c
+++ b/lib/util/util_file.c
@@ -130,7 +130,8 @@ char *fgets_slash(TALLOC_CTX *mem_ctx, char *s2, int maxlen, FILE *f)
 			    if (start_of_line) {
 				    break;
 			    }
-			    /* fall through */
+
+			    FALL_THROUGH;
 		    default:
 			    start_of_line = false;
 			    s[len++] = c;
diff --git a/libgpo/gpo_sec.c b/libgpo/gpo_sec.c
index af73697..98ee8eb 100644
--- a/libgpo/gpo_sec.c
+++ b/libgpo/gpo_sec.c
@@ -47,13 +47,15 @@ static bool gpo_sd_check_agp_object_guid(const struct security_ace_object *objec
 				       &ext_right_apg_guid)) {
 				return true;
 			}
-			/* FALL TROUGH */
+
+			FALL_THROUGH;
 		case SEC_ACE_INHERITED_OBJECT_TYPE_PRESENT:
 			if (GUID_equal(&object->inherited_type.inherited_type,
 				       &ext_right_apg_guid)) {
 				return true;
 			}
-			/* FALL TROUGH */
+
+			FALL_THROUGH;
 		default:
 			break;
 	}
diff --git a/librpc/idl/dcom.idl b/librpc/idl/dcom.idl
index e53d7b8..5559811 100644
--- a/librpc/idl/dcom.idl
+++ b/librpc/idl/dcom.idl
@@ -52,7 +52,7 @@ interface IUnknown
 	[local] WERROR CreateInstance(
 		[in,unique] MInterfacePointer *pUnknown,
 		[in,unique] GUID *iid,
-		[out, iid_is(riid),unique] MInterfacePointer *ppv
+		[out, iid_is(riid),unique] MInterfacePointer **ppv
 		);
 
 	[call_as(CreateInstance)] WERROR RemoteCreateInstance();
diff --git a/librpc/ndr/ndr_cab.c b/librpc/ndr/ndr_cab.c
index 837ed25..c415bfa 100644
--- a/librpc/ndr/ndr_cab.c
+++ b/librpc/ndr/ndr_cab.c
@@ -95,10 +95,13 @@ static uint32_t ndr_cab_compute_checksum(uint8_t *data, uint32_t length, uint32_
 	switch (length % 4) {
 	case 3:
 		ul |= (((uint32_t)(*pb++)) << 16);
+		FALL_THROUGH;
 	case 2:
 		ul |= (((uint32_t)(*pb++)) <<  8);
+		FALL_THROUGH;
 	case 1:
 		ul |= (uint32_t)(*pb++);
+		FALL_THROUGH;
 	default:
 		break;
 	}
diff --git a/nsswitch/pam_winbind.c b/nsswitch/pam_winbind.c
index e14fcfe..63fede8 100644
--- a/nsswitch/pam_winbind.c
+++ b/nsswitch/pam_winbind.c
@@ -2886,7 +2886,8 @@ int pam_sm_acct_mgmt(pam_handle_t *pamh, int flags,


-- 
Samba Shared Repository



More information about the samba-cvs mailing list