]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
rust: remaining rs_ to SC conversions
authorJason Ish <jason.ish@oisf.net>
Mon, 5 May 2025 16:00:27 +0000 (10:00 -0600)
committerVictor Julien <victor@inliniac.net>
Mon, 5 May 2025 19:41:03 +0000 (21:41 +0200)
15 files changed:
rust/src/common.rs
rust/src/core.rs
rust/src/ike/ike.rs
rust/src/krb/krb5.rs
rust/src/ldap/logger.rs
rust/src/ntp/ntp.rs
rust/src/ssh/ssh.rs
rust/src/util.rs
src/app-layer-ike.c
src/app-layer-parser.c
src/app-layer-ssl.c
src/detect-parse.c
src/output.c
src/suricata.c
src/util-print.c

index aa7741f5e1b49c1aba09247e1a009aeb5675e7e4..814fdd2d38b49d93f9dc827be220cd662799e5b8 100644 (file)
@@ -81,7 +81,7 @@ pub fn rust_string_to_c(s: String) -> *mut c_char {
 ///
 /// s must be allocated by rust, using `CString::new`
 #[no_mangle]
-pub unsafe extern "C" fn rs_cstring_free(s: *mut c_char) {
+pub unsafe extern "C" fn SCRustCStringFree(s: *mut c_char) {
     if s.is_null() {
         return;
     }
@@ -102,7 +102,7 @@ pub fn to_hex(input: &[u8]) -> String {
 }
 
 #[no_mangle]
-pub unsafe extern "C" fn rs_to_hex(
+pub unsafe extern "C" fn SCToHex(
     output: *mut u8, out_len: usize, input: *const u8, in_len: usize,
 ) {
     if out_len < 2 * in_len + 1 {
@@ -119,7 +119,7 @@ pub unsafe extern "C" fn rs_to_hex(
 }
 
 #[no_mangle]
-pub unsafe extern "C" fn rs_to_hex_sep(
+pub unsafe extern "C" fn SCToHex_sep(
     output: *mut u8, out_len: usize, sep: u8, input: *const u8, in_len: usize,
 ) {
     if out_len < 3 * in_len {
index 73a67d22a62e0c821ac47c2b92385ca635b9060d..ee4960293eb76a6aa7b31bf4a3f31935ea84067c 100644 (file)
@@ -205,7 +205,7 @@ pub fn init_ffi(context: &'static SuricataContext)
 }
 
 #[no_mangle]
-pub extern "C" fn rs_init(context: &'static SuricataContext)
+pub extern "C" fn SCRustInit(context: &'static SuricataContext)
 {
     init_ffi(context);
 }
index 398fd6e83b379bd534ab8227bc3c4fd7a4a86a8a..07e73583561bdb6f00566034b18cabb82a9255d6 100644 (file)
@@ -365,7 +365,7 @@ export_tx_data_get!(ike_get_tx_data, IKETransaction);
 export_state_data_get!(ike_get_state_data, IKEState);
 
 #[no_mangle]
-pub unsafe extern "C" fn rs_ike_register_parser() {
+pub unsafe extern "C" fn SCRegisterIkeParser() {
     let default_port = CString::new("500").unwrap();
     let parser = RustParser {
         name: PARSER_NAME.as_ptr() as *const std::os::raw::c_char,
index 79f160f29c7aeedc5626ba06d115deccd25cdc61..db0e472b3075d915c3b402304ef6fae4f225a3eb 100644 (file)
@@ -467,7 +467,7 @@ unsafe extern "C" fn krb5_parse_request_tcp(
             // sanity check to avoid memory exhaustion
             if state.defrag_buf_ts.len() + buf.len() > 100000 {
                 SCLogDebug!(
-                    "rs_krb5_parse_request_tcp: TCP buffer exploded {} {}",
+                    "krb5_parse_request_tcp: TCP buffer exploded {} {}",
                     state.defrag_buf_ts.len(),
                     buf.len()
                 );
@@ -491,7 +491,7 @@ unsafe extern "C" fn krb5_parse_request_tcp(
                     return AppLayerResult::ok();
                 }
                 _ => {
-                    SCLogDebug!("rs_krb5_parse_request_tcp: reading record mark failed!");
+                    SCLogDebug!("krb5_parse_request_tcp: reading record mark failed!");
                     return AppLayerResult::err();
                 }
             }
@@ -525,7 +525,7 @@ unsafe extern "C" fn krb5_parse_response_tcp(
             // sanity check to avoid memory exhaustion
             if state.defrag_buf_tc.len() + buf.len() > 100000 {
                 SCLogDebug!(
-                    "rs_krb5_parse_response_tcp: TCP buffer exploded {} {}",
+                    "krb5_parse_response_tcp: TCP buffer exploded {} {}",
                     state.defrag_buf_tc.len(),
                     buf.len()
                 );
index 7ab372d877beda887db698f210197c9851d1dce7..57557f3a40ef3bbbeb8cf67ee6461c3401abd768 100644 (file)
@@ -348,7 +348,7 @@ fn log_controls(controls: &Option<Vec<Control>>, js: &mut JsonBuilder) -> Result
 }
 
 #[no_mangle]
-pub unsafe extern "C" fn rs_ldap_logger_log(
+pub unsafe extern "C" fn SCLdapLoggerLog(
     tx: *mut std::os::raw::c_void, js: &mut JsonBuilder,
 ) -> bool {
     let tx = cast_pointer!(tx, LdapTransaction);
index 468eb68e7ddea3384f7a6e47e5ec0a2369fa9bdc..d547a496255f8b4fedaed69b0fad5c83f463a80d 100644 (file)
@@ -254,7 +254,7 @@ export_state_data_get!(ntp_get_state_data, NTPState);
 const PARSER_NAME: &[u8] = b"ntp\0";
 
 #[no_mangle]
-pub unsafe extern "C" fn rs_register_ntp_parser() {
+pub unsafe extern "C" fn SCRegisterNtpParser() {
     let default_port = CString::new("123").unwrap();
     let parser = RustParser {
         name: PARSER_NAME.as_ptr() as *const std::os::raw::c_char,
index e9ead583776e3c8c691fc2a66ae6423dcc84f5a5..174644d85b49074f2691f98e7068544867978454 100644 (file)
@@ -441,7 +441,7 @@ extern "C" fn ssh_state_tx_free(_state: *mut std::os::raw::c_void, _tx_id: u64)
     //do nothing
 }
 
-unsafe extern "C" fn rs_ssh_parse_request(
+unsafe extern "C" fn ssh_parse_request(
     flow: *const Flow, state: *mut std::os::raw::c_void, pstate: *mut std::os::raw::c_void,
     stream_slice: StreamSlice, _data: *const std::os::raw::c_void,
 ) -> AppLayerResult {
@@ -535,7 +535,7 @@ pub unsafe extern "C" fn SCRegisterSshParser() {
         state_new: ssh_state_new,
         state_free: ssh_state_free,
         tx_free: ssh_state_tx_free,
-        parse_ts: rs_ssh_parse_request,
+        parse_ts: ssh_parse_request,
         parse_tc: ssh_parse_response,
         get_tx_count: ssh_state_get_tx_count,
         get_tx: SCSshStateGetTx,
index d3f76e41eb7c6649d11b9276211d1e5ed89f41ab..56b37ea8531eaf017a355479013044b1025445cf 100644 (file)
@@ -28,7 +28,7 @@ use nom7::multi::many1_count;
 use nom7::IResult;
 
 #[no_mangle]
-pub unsafe extern "C" fn rs_check_utf8(val: *const c_char) -> bool {
+pub unsafe extern "C" fn SCCheckUtf8(val: *const c_char) -> bool {
     CStr::from_ptr(val).to_str().is_ok()
 }
 
index fa8c33cc8094e52a2a97ad7072fcab49b6c78635..f11ecdd356da930a5412fa19a02917a5c7151cd9 100644 (file)
@@ -39,7 +39,7 @@
 
 void RegisterIKEParsers(void)
 {
-    rs_ike_register_parser();
+    SCRegisterIkeParser();
 #ifdef UNITTESTS
     AppLayerParserRegisterProtocolUnittests(IPPROTO_UDP, ALPROTO_IKE, IKEParserRegisterTests);
 #endif
index cbf97e23469aecd29767c2a6ce9ae1b057428b10..75845c4a5180db93ee79003aca5b84848662fadb 100644 (file)
@@ -1787,7 +1787,7 @@ void AppLayerParserRegisterProtocolParsers(void)
     RegisterDNP3Parsers();
     RegisterNFSTCPParsers();
     RegisterNFSUDPParsers();
-    rs_register_ntp_parser();
+    SCRegisterNtpParser();
     RegisterTFTPParsers();
     RegisterIKEParsers();
     SCRegisterKrb5Parser();
index 10c9dce7cae47ad7ba4e02c24d8eb72e17ff6388..ff8a11e3fae639abad2b5d16e4baa041e1d12c99 100644 (file)
@@ -532,7 +532,7 @@ static inline int TlsDecodeHSCertificateFingerprint(
 
     uint8_t hash[SC_SHA1_LEN];
     if (SCSha1HashBuffer(input, cert_len, hash, sizeof(hash)) == 1) {
-        rs_to_hex_sep(
+        SCToHex_sep(
                 (uint8_t *)connp->cert0_fingerprint, SHA1_STRING_LENGTH, ':', hash, SC_SHA1_LEN);
     }
     return 0;
@@ -2973,7 +2973,7 @@ static void SSLStateCertSANFree(SSLStateConnp *connp)
 {
     if (connp->cert0_sans) {
         for (uint16_t i = 0; i < connp->cert0_sans_len; i++) {
-            rs_cstring_free(connp->cert0_sans[i]);
+            SCRustCStringFree(connp->cert0_sans[i]);
         }
         SCFree(connp->cert0_sans);
     }
@@ -2989,11 +2989,11 @@ static void SSLStateFree(void *p)
     SSLCertsChain *item;
 
     if (ssl_state->client_connp.cert0_subject)
-        rs_cstring_free(ssl_state->client_connp.cert0_subject);
+        SCRustCStringFree(ssl_state->client_connp.cert0_subject);
     if (ssl_state->client_connp.cert0_issuerdn)
-        rs_cstring_free(ssl_state->client_connp.cert0_issuerdn);
+        SCRustCStringFree(ssl_state->client_connp.cert0_issuerdn);
     if (ssl_state->client_connp.cert0_serial)
-        rs_cstring_free(ssl_state->client_connp.cert0_serial);
+        SCRustCStringFree(ssl_state->client_connp.cert0_serial);
     if (ssl_state->client_connp.cert0_fingerprint)
         SCFree(ssl_state->client_connp.cert0_fingerprint);
     if (ssl_state->client_connp.sni)
@@ -3004,11 +3004,11 @@ static void SSLStateFree(void *p)
         SCFree(ssl_state->client_connp.hs_buffer);
 
     if (ssl_state->server_connp.cert0_subject)
-        rs_cstring_free(ssl_state->server_connp.cert0_subject);
+        SCRustCStringFree(ssl_state->server_connp.cert0_subject);
     if (ssl_state->server_connp.cert0_issuerdn)
-        rs_cstring_free(ssl_state->server_connp.cert0_issuerdn);
+        SCRustCStringFree(ssl_state->server_connp.cert0_issuerdn);
     if (ssl_state->server_connp.cert0_serial)
-        rs_cstring_free(ssl_state->server_connp.cert0_serial);
+        SCRustCStringFree(ssl_state->server_connp.cert0_serial);
     if (ssl_state->server_connp.cert0_fingerprint)
         SCFree(ssl_state->server_connp.cert0_fingerprint);
     if (ssl_state->server_connp.sni)
index ad7696c731c2090e6675330c979570d379c68825..01c5efd64aeb9c523d988520ddd9ba7373771d7c 100644 (file)
@@ -1828,7 +1828,7 @@ static int SigParse(DetectEngineCtx *de_ctx, Signature *s, const char *sigstr,
 {
     SCEnter();
 
-    if (!rs_check_utf8(sigstr)) {
+    if (!SCCheckUtf8(sigstr)) {
         SCLogError("rule is not valid UTF-8");
         SCReturnInt(-1);
     }
index c1fa74db50bea41237f5c92cd9b5242c04489bfa..d03cdcaa593d56e4ff77eb48d2f2d0b1b5cb0c08 100644 (file)
@@ -924,8 +924,7 @@ void OutputRegisterRootLoggers(void)
             ALPROTO_PGSQL, (EveJsonSimpleTxLogFunc)JsonPgsqlAddMetadata, NULL);
     RegisterSimpleJsonApplayerLogger(
             ALPROTO_WEBSOCKET, (EveJsonSimpleTxLogFunc)SCWebSocketLoggerLog, NULL);
-    RegisterSimpleJsonApplayerLogger(
-            ALPROTO_LDAP, (EveJsonSimpleTxLogFunc)rs_ldap_logger_log, NULL);
+    RegisterSimpleJsonApplayerLogger(ALPROTO_LDAP, (EveJsonSimpleTxLogFunc)SCLdapLoggerLog, NULL);
     RegisterSimpleJsonApplayerLogger(ALPROTO_DOH2, (EveJsonSimpleTxLogFunc)AlertJsonDoh2, NULL);
     RegisterSimpleJsonApplayerLogger(
             ALPROTO_TEMPLATE, (EveJsonSimpleTxLogFunc)SCTemplateLoggerLog, NULL);
index 9b3099149eba81ff8f14421462bc1ac418eca8ab..6e9a0dacf44ba6072971e03281fa2ddf0c6f9109 100644 (file)
@@ -2879,7 +2879,7 @@ void SuricataMainLoop(void)
 
 int InitGlobal(void)
 {
-    rs_init(&suricata_context);
+    SCRustInit(&suricata_context);
 
     SC_ATOMIC_INIT(engine_stage);
 
index 77d88faa64dde721e5a71edb1c87ea274217ca16..b7f33583d8fd5ba907c5f6fb28bd68988693eb5f 100644 (file)
@@ -255,5 +255,5 @@ const char *PrintInet(int af, const void *src, char *dst, socklen_t size)
 void PrintHexString(char *str, size_t size, uint8_t *buf, size_t buf_len)
 {
     DEBUG_VALIDATE_BUG_ON(size < 2 * buf_len);
-    rs_to_hex((uint8_t *)str, size, buf, buf_len);
+    SCToHex((uint8_t *)str, size, buf, buf_len);
 }