[Asterisk-code-review] media cache: Mark cache entry stale of cache file is removed (asterisk[master])

[Sean Bright]

Sean Bright has uploaded a new change for review. ( https://gerrit.asterisk.org/5108 )

Change subject: media_cache: Mark cache entry stale of cache file is removed
......................................................................

media_cache: Mark cache entry stale of cache file is removed

In the event that a cache file is removed out from under us, we should
treat the cache entry as stale and force a fresh retrieval.

ASTERISK-26774 #close
Reported by: Igor Gamayunov

Change-Id: I3b1bd0c999d59d18664ef73a29823bc5b431dc52
---
M main/media_cache.c
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/08/5108/1

diff --git a/main/media_cache.c b/main/media_cache.c
index 7be3588..2592f35 100644
--- a/main/media_cache.c
+++ b/main/media_cache.c
@@ -224,7 +224,8 @@
 	 */
 	bucket_file = ao2_find(media_cache, uri, OBJ_SEARCH_KEY | OBJ_NOLOCK);
 	if (bucket_file) {
-		if (!ast_bucket_file_is_stale(bucket_file)) {
+		if (!ast_bucket_file_is_stale(bucket_file)
+			&& !access(bucket_file->path, R_OK)) {
 			ast_copy_string(file_path, bucket_file->path, len);
 			if ((ext = strrchr(file_path, '.'))) {
 				*ext = '\0';

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3b1bd0c999d59d18664ef73a29823bc5b431dc52
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>