sdp: fix logging medias

As introduced by bff790b Also handles errors in the caller Ticket: 6994
jufajardini · Apr 30, 2024 · c53e9ac · c53e9ac
1 parent df8568e
commit c53e9ac
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/rust/src/sdp/logger.rs b/rust/src/sdp/logger.rs
@@ -107,8 +107,8 @@ fn log_media_description(
             }
             js.close()?;
         }
+        js.close()?;
     }
-    js.close()?;
 
     Ok(())
 }

diff --git a/rust/src/sip/log.rs b/rust/src/sip/log.rs
@@ -30,7 +30,7 @@ fn log(tx: &SIPTransaction, js: &mut JsonBuilder) -> Result<(), JsonError> {
             .set_string("version", &req.version)?;
 
         if let Some(sdp_body) = &req.body {
-            let _ = sdp_log(sdp_body, js);
+            sdp_log(sdp_body, js)?;
         }
     }
 
@@ -43,7 +43,7 @@ fn log(tx: &SIPTransaction, js: &mut JsonBuilder) -> Result<(), JsonError> {
             .set_string("code", &resp.code)?
             .set_string("reason", &resp.reason)?;
         if let Some(sdp_body) = &resp.body {
-            let _ = sdp_log(sdp_body, js);
+            sdp_log(sdp_body, js)?;
         }
     }