detect/files: reuse AppLayerParserSupportsFiles

rather than relisting the protocols

src/detect-file-data.c

@@ -197,11 +197,8 @@ static int DetectFiledataSetup (DetectEngineCtx *de_ctx, Signature *s, const cha
     SCEnter();
 
     if (!DetectProtoContainsProto(&s->proto, IPPROTO_TCP) ||
-            (s->alproto != ALPROTO_UNKNOWN && s->alproto != ALPROTO_HTTP1 &&
+            (s->alproto != ALPROTO_UNKNOWN &&
-                    s->alproto != ALPROTO_SMTP && s->alproto != ALPROTO_SMB &&
+                    !AppLayerParserSupportsFiles(IPPROTO_TCP, s->alproto))) {
-                    s->alproto != ALPROTO_HTTP2 && s->alproto != ALPROTO_FTP &&
-                    s->alproto != ALPROTO_FTPDATA && s->alproto != ALPROTO_HTTP &&
-                    s->alproto != ALPROTO_NFS)) {
         SCLogError("rule contains conflicting keywords.");
         return -1;
     }