]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
sdp: add sdp.media.media sticky buffer
authorGiuseppe Longo <giuseppe@glongo.it>
Wed, 2 Oct 2024 08:10:39 +0000 (10:10 +0200)
committerVictor Julien <victor@inliniac.net>
Wed, 2 Apr 2025 20:36:16 +0000 (22:36 +0200)
This adds a sticky (multi) buffer to match the "Media" subfield of the
"Media description" field in both requests and responses.

Ticket #7291

rust/src/sdp/detect.rs

index 97ed4f811a6b26c4bc7b5181f5c58df0c7d3d465..2e857145f96e6181eae41096e8ded0cd563ac380 100644 (file)
@@ -40,6 +40,7 @@ static mut G_SDP_REPEAT_TIME_BUFFER_ID: c_int = 0;
 static mut G_SDP_TIMEZONE_BUFFER_ID: c_int = 0;
 static mut G_SDP_ENCRYPTION_KEY_BUFFER_ID: c_int = 0;
 static mut G_SDP_ATTRIBUTE_BUFFER_ID: c_int = 0;
+static mut G_SDP_MEDIA_DESC_MEDIA_BUFFER_ID: c_int = 0;
 
 unsafe extern "C" fn sdp_session_name_setup(
     de: *mut c_void, s: *mut c_void, _raw: *const std::os::raw::c_char,
@@ -672,6 +673,58 @@ unsafe extern "C" fn sip_attribute_get_data(
     false
 }
 
+unsafe extern "C" fn sdp_media_desc_media_setup(
+    de: *mut c_void, s: *mut c_void, _raw: *const std::os::raw::c_char,
+) -> c_int {
+    if DetectSignatureSetAppProto(s, ALPROTO_SIP) != 0 {
+        return -1;
+    }
+    if DetectBufferSetActiveList(de, s, G_SDP_MEDIA_DESC_MEDIA_BUFFER_ID) < 0 {
+        return -1;
+    }
+    return 0;
+}
+
+unsafe extern "C" fn sdp_media_desc_media_get(
+    de: *mut c_void, transforms: *const c_void, flow: *const c_void, flow_flags: u8,
+    tx: *const c_void, list_id: c_int, local_id: u32,
+) -> *mut c_void {
+    return DetectHelperGetMultiData(
+        de,
+        transforms,
+        flow,
+        flow_flags,
+        tx,
+        list_id,
+        local_id,
+        sip_media_desc_media_get_data,
+    );
+}
+
+unsafe extern "C" fn sip_media_desc_media_get_data(
+    tx: *const c_void, flow_flags: u8, local_id: u32, buffer: *mut *const u8, buffer_len: *mut u32,
+) -> bool {
+    let tx = cast_pointer!(tx, SIPTransaction);
+    let direction = flow_flags.into();
+    let sdp_option = match direction {
+        Direction::ToServer => tx.request.as_ref().and_then(|req| req.body.as_ref()),
+        Direction::ToClient => tx.response.as_ref().and_then(|resp| resp.body.as_ref()),
+    };
+    if let Some(sdp) = sdp_option {
+        if let Some(ref m) = sdp.media_description {
+            if (local_id as usize) < m.len() {
+                let val = &m[local_id as usize].media;
+                *buffer = val.as_ptr();
+                *buffer_len = val.len() as u32;
+                return true;
+            }
+        }
+    }
+    *buffer = ptr::null();
+    *buffer_len = 0;
+    false
+}
+
 #[no_mangle]
 pub unsafe extern "C" fn ScDetectSdpRegister() {
     let kw = SCSigTableElmt {
@@ -916,4 +969,23 @@ pub unsafe extern "C" fn ScDetectSdpRegister() {
         true,
         sdp_attribute_get,
     );
+    let kw = SCSigTableElmt {
+        name: b"sdp.media.media\0".as_ptr() as *const libc::c_char,
+        desc: b"sticky buffer to match on the SDP media subfield of the media_description field\0"
+            .as_ptr() as *const libc::c_char,
+        url: b"/rules/sdp-keywords.html#media-description-media\0".as_ptr() as *const libc::c_char,
+        Setup: sdp_media_desc_media_setup,
+        flags: SIGMATCH_NOOPT,
+        AppLayerTxMatch: None,
+        Free: None,
+    };
+    let _ = DetectHelperKeywordRegister(&kw);
+    G_SDP_MEDIA_DESC_MEDIA_BUFFER_ID = DetectHelperMultiBufferMpmRegister(
+        b"sdp.media.media\0".as_ptr() as *const libc::c_char,
+        b"sdp.media.media\0".as_ptr() as *const libc::c_char,
+        ALPROTO_SIP,
+        true,
+        true,
+        sdp_media_desc_media_get,
+    );
 }