In message
[x265] x265 crashes/gets stuck when giving more than '--slices 16'
there is description of crash. You can reproduce this crash by:
x265 --slices 17 ducks_take_off_1080p50.y4m out.hevc

This small patch avoid the crash. Please review.

# HG changeset patch
# User Ma0 <[email protected]>
# Date 1489188353 -3600
#      Sat Mar 11 00:25:53 2017 +0100
# Node ID b5daf53bd011d4c0e08880de4fa46310f1111e32
# Parent  88fd9082764c7d7b4668e30763a93215980efd70
fix maxSlices limit checking

diff -r 88fd9082764c -r b5daf53bd011 source/encoder/encoder.cpp
--- a/source/encoder/encoder.cpp    Wed Mar 08 17:15:59 2017 +0530
+++ b/source/encoder/encoder.cpp    Sat Mar 11 00:25:53 2017 +0100
@@ -2294,7 +2294,7 @@
 
     const uint32_t numRows = (p->sourceHeight + p->maxCUSize - 1) / 
p->maxCUSize;
     const uint32_t slicesLimit = X265_MIN(numRows, NALList::MAX_NAL_UNITS - 1);
-    if (p->maxSlices > numRows)
+    if (p->maxSlices > slicesLimit)
     {
         x265_log(p, X265_LOG_WARNING, "maxSlices can not be more than 
min(rows, MAX_NAL_UNITS-1), force set to %d\n", slicesLimit);
         p->maxSlices = slicesLimit;

# HG changeset patch
# User Ma0 <[email protected]>
# Date 1489188353 -3600
#      Sat Mar 11 00:25:53 2017 +0100
# Node ID b5daf53bd011d4c0e08880de4fa46310f1111e32
# Parent  88fd9082764c7d7b4668e30763a93215980efd70
fix maxSlices limit checking

diff -r 88fd9082764c -r b5daf53bd011 source/encoder/encoder.cpp
--- a/source/encoder/encoder.cpp        Wed Mar 08 17:15:59 2017 +0530
+++ b/source/encoder/encoder.cpp        Sat Mar 11 00:25:53 2017 +0100
@@ -2294,7 +2294,7 @@
 
     const uint32_t numRows = (p->sourceHeight + p->maxCUSize - 1) / 
p->maxCUSize;
     const uint32_t slicesLimit = X265_MIN(numRows, NALList::MAX_NAL_UNITS - 1);
-    if (p->maxSlices > numRows)
+    if (p->maxSlices > slicesLimit)
     {
         x265_log(p, X265_LOG_WARNING, "maxSlices can not be more than 
min(rows, MAX_NAL_UNITS-1), force set to %d\n", slicesLimit);
         p->maxSlices = slicesLimit;
_______________________________________________
x265-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/x265-devel

Reply via email to