[Asterisk-code-review] format: Register format-attribute module with cached formats. (asterisk[master])

[Joshua Colp]

Joshua Colp has posted comments on this change.

Change subject: format: Register format-attribute module with cached formats.
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/1600/1/main/format.c
File main/format.c:

Line 162: 	cached = ast_format_cache_get(codec);
        : 	if (cached) {
        : 		cached->interface = format_interface->interface;
        : 	}
This is dangerous. Formats are immutable once created and have no protection around them. If a format is used during this time there is a chance of a crash as it is manipulated.


-- 
To view, visit https://gerrit.asterisk.org/1600
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ea1f0483e5261e2a050112e4ebdfc22057d1354
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-HasComments: Yes