lib/fuzzing and librpc: Do not generate fuzzers for pointless targets
authorAndrew Bartlett <abartlet@samba.org>
Fri, 13 Dec 2019 02:34:34 +0000 (15:34 +1300)
committerAndrew Bartlett <abartlet@samba.org>
Wed, 18 Dec 2019 06:39:26 +0000 (06:39 +0000)
We need to focus the fuzzing effort on reachable code, and these IDL
are just historical artifacts, many are entirely [todo] and have
no samba client nor server.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Gary Lockyer <gary@catalyst.net.nz>
lib/fuzzing/wscript_build
librpc/idl/wscript_build

index e77eea88df56a76c4751c2396283e90e37284255..75c41ac83f44bedf59479e997bf0e7aeb1b9f969 100644 (file)
@@ -97,7 +97,6 @@ bld.SAMBA_NDR_FUZZ('FileServerVssAgent') # fsvrp.idl
 bld.SAMBA_NDR_FUZZ('lsarpc') # lsa.idl
 bld.SAMBA_NDR_FUZZ('netdfs') # dfs.idl
 bld.SAMBA_NDR_FUZZ('nfs4acl_interface') # nfs4acl.idl
-bld.SAMBA_NDR_FUZZ('ObjectRpcBaseTypes') # orpc.idl
 bld.SAMBA_NDR_FUZZ('rpcecho') # echo.idl
 
 # quota.idl
@@ -113,8 +112,6 @@ bld.SAMBA_NDR_FUZZ('sparse')
 bld.SAMBA_NDR_FUZZ('resiliency')
 bld.SAMBA_NDR_FUZZ('trim')
 
-# Skipped: dsbackup (all todo)
-
 # WMI tables
 bld.SAMBA_NDR_FUZZ('IWbemClassObject')
 bld.SAMBA_NDR_FUZZ('IWbemServices')
index 5dda944ca7165864af518c2f8f130551f9180f84..830308491dac717bc89d8ff5a1092524d89055bc 100644 (file)
@@ -71,16 +71,22 @@ bld.SAMBA_PIDL_LIST('PIDL',
                     generate_fuzzers=False)
 
 # DCE/RPC protocols which Samba does not implement a client or server
-# for
+# for.  We don't generate a fuzzer for these as they are unreachable
+#
+# Do not include IDL with public structures in this list as we want to
+# fuzz those
 
 bld.SAMBA_PIDL_LIST('PIDL',
                     '''
                     audiosrv.idl
+                    dbgidl.idl
+                    dsbackup.idl
                     efs.idl
                     frstrans.idl
                     frsrpc.idl
                     keysvc.idl
                     msgsvc.idl
+                    orpc.idl
                     policyagent.idl
                     rot.idl
                     scerpc.idl
@@ -89,20 +95,9 @@ bld.SAMBA_PIDL_LIST('PIDL',
                     wzcsvc.idl
                     ''',
                     options='--header --ndr-parser',
+                    generate_fuzzers=False,
                     output_dir='../gen_ndr')
 
-# The interface names here are not the same as the IDL name, so the
-# auto-genration of the fuzzer fails to link
-
-bld.SAMBA_PIDL_LIST('PIDL',
-                    '''
-                    dsbackup.idl
-                    orpc.idl
-                    ''',
-                    options='--header --ndr-parser',
-                    output_dir='../gen_ndr',
-                    generate_fuzzers=False)
-
 # Non-DCE/RPC protocols encoded in IDL for Samba or helper IDLs for
 # DCE/RPC protocols (eg defining constands or structures but not
 # functions)
@@ -110,7 +105,6 @@ bld.SAMBA_PIDL_LIST('PIDL',
                     '''
                     bkupblobs.idl
                     cab.idl
-                    dbgidl.idl
                     file_id.idl
                     fscc.idl
                     fsrvp_state.idl