Attention is currently required from: fixeria.
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/28756 )

Change subject: trxcon: use trxcon->fi as the context in L1CTL logging
......................................................................


Patch Set 7:

(1 comment)

File src/host/trxcon/src/trxcon.c:

https://gerrit.osmocom.org/c/osmocom-bb/+/28756/comment/59f1e9fc_300072bb
PS7, Line 392:  l1c->log_prefix = talloc_strdup(l1c, trxcon->log_prefix);
just to be sure, is it possible that there may be a leak here? 
osmo_talloc_replace?



--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/28756
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I7b923cb21638e6afc38deb7766955b43e49b60bb
Gerrit-Change-Number: 28756
Gerrit-PatchSet: 7
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Mon, 25 Jul 2022 17:50:02 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to