:: Re: [Frei0r] Release ?
Top Page
Delete this message
Reply to this message
Author: salsaman
Date:  
To: Brian Matherly
CC: Minimalistic plugin API for video effects
Subject: Re: [Frei0r] Release ?
Confirmed fixed.

Thanks :D



Salsaman.





http://lives-video.com
https://www.openhub.net/accounts/salsaman

On Tue, Mar 15, 2016 at 11:15 PM, Brian Matherly <pez4brian@???>
wrote:

> Fix submitted here:
> https://github.com/ddennedy/frei0r/pull/6
>
> ~BM
>
> ------------------------------
> *From:* salsaman <salsaman@???>
> *To:* Brian Matherly <pez4brian@???>
> *Cc:* Minimalistic plugin API for video effects <frei0r@???>
> *Sent:* Tuesday, March 15, 2016 8:32 PM
>
> *Subject:* Re: [Frei0r] Release ?
>
> Ah. makes sense:
>
> void GradientLut::setDepth(size_t depth) {
>     lut.resize(depth);
> }

>
> lust.size gets set to zero, then in operator[]:
>
>
>     if(index >= size) {
>         index = size - 1;
>     }

>
>
> index gets set to -1.
>
>
>
>
>
>
>
>
> http://lives-video.com
> https://www.openhub.net/accounts/salsaman
>
> On Tue, Mar 15, 2016 at 10:30 PM, salsaman <salsaman@???> wrote:
>
> It seems to crash when "levels" is set to zero.
>
>
>
>
>
> http://lives-video.com
> https://www.openhub.net/accounts/salsaman
>
> On Tue, Mar 15, 2016 at 10:08 PM, Brian Matherly <pez4brian@???>
> wrote:
>
> It isn't an array. It's an operator[] on the GradientLut class.
>
> /**
>  * Get a color value for a given position in the table.
>  * The index in the table will be calculated proportionally to the depth
> of the
>  * LUT.
>  */
> const GradientLut::Color& GradientLut::operator[](double pos) const {
>     unsigned int size = lut.size();
>     unsigned int index = (double)size * pos;
>     if(index >= size) {
>         index = size - 1;
>     }
>     return lut[index];
> };

>
> There is range protection right before the array is indexed.
>
> ~BM
>
>
> ------------------------------
> *From:* salsaman <salsaman@???>
> *To:* Brian Matherly <pez4brian@???>; Minimalistic plugin API for
> video effects <frei0r@???>
> *Sent:* Tuesday, March 15, 2016 7:54 PM
>
> *Subject:* Re: [Frei0r] Release ?
>
> inline void Ndvi::setColor(uint8_t *sample, double index)
> {
>     double pos = N2P(index);
>     const GradientLut::Color& falseColor = gradient[pos];

>
>
>
> I don't think using a double as an array index is a very good idea.
>
>
> Regards,
> Salsaman.
>
>
>
>
>
> http://lives-video.com
> https://www.openhub.net/accounts/salsaman
>
> On Tue, Mar 15, 2016 at 9:46 PM, Brian Matherly <pez4brian@???>
> wrote:
>
> I'm willing to look at it.
>
> The only way I can see that this would happen is if the output image
> pointer is corrupted.
>
> Can you provide steps to reproduce?
>
> ~Brian
>
>
> ------------------------------
> *From:* salsaman <salsaman@???>
> *To:* Minimalistic plugin API for video effects <frei0r@???>
> *Sent:* Tuesday, March 15, 2016 7:03 PM
> *Subject:* Re: [Frei0r] Release ?
>
> OK, just one bug found:
>
> Program received signal SIGSEGV, Segmentation fault.
>
> Ndvi::setColor (index=<optimized out>, sample=0x3cc4270 "",
> this=0x38a3e70) at filter/ndvi/ndvi.cpp:236
> 236        sample[0] = falseColor.r;

>
>
>
> Somebody should fix that before the release.
>
>
>
>
> Regards,
> Salsaman.
>
>
>
>
>
>
>
>
> http://lives-video.com
> https://www.openhub.net/accounts/salsaman
>
> On Tue, Mar 15, 2016 at 7:23 PM, Dan Dennedy <dan@???> wrote:
>
> On Tue, Mar 15, 2016 at 3:20 PM salsaman <salsaman@???> wrote:
>
> Alright then, just give me a couple of days to check the git master.
>
>
> I think that is fine. I will plan to bump version and make dist tarball on
> the weekend.
>
>
> Regards,
> Salsaman.
>
>
> http://lives-video.com
> https://www.openhub.net/accounts/salsaman
>
> On Tue, Mar 15, 2016 at 7:14 PM, Dan Dennedy <dan@???> wrote:
>
> On Tue, Mar 15, 2016 at 2:24 PM salsaman <salsaman@???> wrote:
>
> Can you let us test the build before releasing it.
>
>
> No, sorry. It has already been tested some by apps such as Shotcut that
> use the git master. That is all the testing I volunteer to do. If you
> wanted it tested, you should have been doing that. It is past due for a
> release.
>
>
>
> Salsaman.
>
>
>
> http://lives-video.com
> https://www.openhub.net/accounts/salsaman
>
> On Tue, Mar 15, 2016 at 5:18 PM, Dan Dennedy <dan@???> wrote:
>
> On Tue, Mar 15, 2016 at 1:37 AM jb <jb@???> wrote:
>
> Le 27.02.16 09:52, Jb a écrit :
> > Hi all,
> >
> > Would it be possible to get a new frei0r release? There were several
> > fixes in the recent months, and the cairo mixers transparency issue,
> > that was fixed by Dan in november hits many Kdenlive users.
> >
> > A release would really help to spread an updated frei0r to end users.
>
> Ping... is anyone still active in the frei0r maintainers team ? We have
> a Kdenlive release in april and it would have been really nice to have a
> coordinated frei0r release.
>
>
>
> Jaromil maintains the web server and has been release manager. I just
> emailed him, and he responded, but he has not been monitoring the mailing
> list. I am going to make the release and send it to him, and he will host
> it.
>
>
> _______________________________________________
> Frei0r mailing list - http://frei0r.dyne.org
> Free video plugins, minimal and cross-platform.
> https://mailinglists.dyne.org/cgi-bin/mailman/listinfo/frei0r
>
>
>
> _______________________________________________
> Frei0r mailing list - http://frei0r.dyne.org
> Free video plugins, minimal and cross-platform.
> https://mailinglists.dyne.org/cgi-bin/mailman/listinfo/frei0r
>
>
>
> _______________________________________________
> Frei0r mailing list - http://frei0r.dyne.org
> Free video plugins, minimal and cross-platform.
> https://mailinglists.dyne.org/cgi-bin/mailman/listinfo/frei0r
>
>
>
>
>
>
>
>
>