Dirk Farin
Dirk Farin
I've looked at the file. The depth map is stored with JPEG compression. Now, libheif just added support for JPEG compressed images in the HEIF, but this file uses lossless...
This is marked as draft since 2022. Do you want to keep this open or should we close it?
We might be able to omit that limit for grid images (total image size) because with the planned ROI decoding, it would be perfectly fine to have a huge image...
Image Grid has a maximum of 256 tile rows/columns, stored in uint16.
I suggest that we limit the total number of tiles instead of width/height separately, as was recently implemented in https://github.com/strukturag/libheif/pull/1204 I.e. `MAX_NUMBER_OF_TILES` = 32768*32768;
> > I.e. `MAX_NUMBER_OF_TILES` = 32768*32768; > > Is this the limit value you want to use? Essentially allowing number of tiles to equal number of pixels? I just used...
Yes. And for huge images, the user does not even have to increase the limit if he is accessing the image only in small ROIs that are below the limit....
Looks good. Thank you.
Even though the input file has a wrong property order, after this refactoring 0b236060d0585d744e26e2f745cb18419a38ccb8, it now reads "correctly".
Security limits are now passed to the plugins (944ac3f92e39bf4f664e08b7515a86f65aa36ad4). However, this currently only checks the allocation of the output `heif_image`. Ideally, we should check the image size of the compressed...