<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta http-equiv="Content-Type" content="text/xhtml; charset=utf-8">
<style>
div.markdown { white-space: normal; }
div.plaintext { white-space: pre-wrap; }
body { font-family: sans-serif; }
h1 { font-size: 1.4em; }
h2 { font-size: 1.2em; }
h3 { font-size: 1.1em; }
blockquote { margin: 0 0 5px; padding-left: 5px; border-left: 2px solid #777777; color: #777777; }
blockquote blockquote { border-left-color: #999999; color: #999999; }
blockquote blockquote blockquote { border-left-color: #BBBBBB; color: #BBBBBB; }
a { color: #3983C4 }
blockquote a { color: #777777; }
blockquote blockquote a { color: #999999; }
blockquote blockquote blockquote a { color: #BBBBBB; }
code { margin: 0; padding: 0 0.4em; border-radius: 3px; background-color: #F7F7F7; }
pre > code { padding: 0px; }
math[display="inline"] > mrow { padding:5px; }
div.footnotes li p { margin: 0.2em 0; }
</style>
</head>
<body>
<div class="markdown">
<p dir="auto">On 2 Oct 2016, at 11:36, Andrey wrote:</p>

<p dir="auto"></div>
<div class="plaintext"><blockquote><div dir="auto">source.c, function source_reserve() :
</div><div dir="auto">Function avl_tree_wlock(global.source_tree) call must be placed before do-while, not inside the cycle.
</div></blockquote></div>
<div class="markdown"></p>

<p dir="auto">I do not think this makes any difference, this is a <code>do {} while (0)</code>, so it will only execute once anyway.</p>

<p dir="auto"></div>
<div class="plaintext"><blockquote><div dir="auto">
</div><div dir="auto">My correction in block _DIGI:
</div><div dir="auto">
</div><div dir="auto">#define _DIGI
</div><div dir="auto">
</div><div dir="auto">...
</div><div dir="auto">    if(mount[0] != '/')
</div><div dir="auto">        ICECAST_LOG_WARN("Source at \"%s\" does not start with '/', clients will be "
</div><div dir="auto">                "unable to connect", mount);
</div><div dir="auto">
</div><div dir="auto">#ifdef _DIGI
</div><div dir="auto">    avl_tree_wlock (global.source_tree);
</div><div dir="auto">    do
</div><div dir="auto">    {
</div><div dir="auto">#else
</div><div dir="auto">    do
</div><div dir="auto">    {
</div><div dir="auto">        avl_tree_wlock (global.source_tree);
</div><div dir="auto">#endif
</div><div dir="auto">        src = source_find_mount_raw (mount);
</div><div dir="auto">        if (src)
</div><div dir="auto">        {
</div><div dir="auto">            src = NULL;
</div><div dir="auto">            break;
</div><div dir="auto">        }
</div><div dir="auto">...
</div><div dir="auto">_______________________________________________
</div><div dir="auto">Icecast-dev mailing list
</div><div dir="auto">Icecast-dev@xiph.org
</div><div dir="auto"><a href="http://lists.xiph.org/mailman/listinfo/icecast-dev">http://lists.xiph.org/mailman/listinfo/icecast-dev</a>
</div></blockquote></div>
<div class="markdown"></p>
</div>

</body>
</html>