<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body style="overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;">Hi all,<div><p style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);">Just a friendly ping on this patch. Is there any feedback or anything I can do to help get it reviewed?</p><p style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);">Thanks</p><div><blockquote type="cite"><div>2025年6月11日 09:55，yuygfgg <leoyuygfgg@gmail.com> 写道：</div><br class="Apple-interchange-newline"><div><div>In Encoder::fetchStats, the size check for the stats buffer was<br>incorrectly using sizeof(stats) on the x265_stats pointer. This<br>evaluates to the size of a pointer, not the size of the structure<br>it points to.<br><br>This commit changes the check to sizeof(*stats) to correctly<br>compare against the size of the x265_stats structure.<br><br>Signed-off-by: yuygfgg <leoyuygfgg@gmail.com><br>---<br> source/encoder/encoder.cpp | 2 +-<br> 1 file changed, 1 insertion(+), 1 deletion(-)<br><br>diff --git a/source/encoder/encoder.cpp b/source/encoder/encoder.cpp<br>index cf2e95988..be898cbbf 100644<br>--- a/source/encoder/encoder.cpp<br>+++ b/source/encoder/encoder.cpp<br>@@ -2963,7 +2963,7 @@ void Encoder::printSummary()<br><br> void Encoder::fetchStats(x265_stats *stats, size_t statsSizeBytes, int layer)<br> {<br>- if (statsSizeBytes >= sizeof(stats))<br>+ if (statsSizeBytes >= sizeof(*stats))<br> {<br> stats->globalPsnrY = m_analyzeAll[layer].m_psnrSumY;<br> stats->globalPsnrU = m_analyzeAll[layer].m_psnrSumU;<br>-- <br>2.49.0<br><br></div></div></blockquote></div><br></div></body></html>