Fix T97356: Crash when adding adjustment layer strip
Crash was caused by incorrectly assuming, that rendered strip is meta when editing code in bulk, but this wasn't true and strip had no channels initialized, which caused crash on NULL dereference. Correct approach is to find list of channels where on same level as rendered strip, similar to how `SEQ_get_seqbase_by_seq` function works for list of strips.
This commit is contained in:
parent
b67ccc333a
commit
3cef9ebaf8
Notes:
blender-bot
2023-02-14 06:47:29 +01:00
Referenced by commit 38a4d96a90
, Fix T97507: Crash when deleting adjustment layer
Referenced by issue #97507, Crash when adding multiple Adjustment layers and then deleting one of them
Referenced by issue #97356, Regression: Crash on changing view type with adjustment layer strips
|
@ -15,6 +15,7 @@ struct Editing;
|
|||
struct ListBase;
|
||||
struct Scene;
|
||||
struct SeqTimelineChannel;
|
||||
struct Sequence;
|
||||
|
||||
struct ListBase *SEQ_channels_displayed_get(struct Editing *ed);
|
||||
void SEQ_channels_displayed_set(struct Editing *ed, struct ListBase *channels);
|
||||
|
@ -28,6 +29,7 @@ char *SEQ_channel_name_get(struct ListBase *channels, const int channel_index);
|
|||
bool SEQ_channel_is_locked(const struct SeqTimelineChannel *channel);
|
||||
bool SEQ_channel_is_muted(const struct SeqTimelineChannel *channel);
|
||||
int SEQ_channel_index_get(const struct SeqTimelineChannel *channel);
|
||||
ListBase *SEQ_get_channels_by_seq(struct ListBase *seqbase, const struct Sequence *seq);
|
||||
|
||||
#ifdef __cplusplus
|
||||
}
|
||||
|
|
|
@ -81,3 +81,19 @@ bool SEQ_channel_is_muted(const SeqTimelineChannel *channel)
|
|||
{
|
||||
return (channel->flag & SEQ_CHANNEL_MUTE) != 0;
|
||||
}
|
||||
|
||||
ListBase *SEQ_get_channels_by_seq(ListBase *seqbase, const Sequence *seq)
|
||||
{
|
||||
ListBase *lb = NULL;
|
||||
|
||||
LISTBASE_FOREACH (Sequence *, iseq, seqbase) {
|
||||
if (seq == iseq) {
|
||||
return seqbase;
|
||||
}
|
||||
if ((lb = SEQ_get_channels_by_seq(&iseq->seqbase, seq))) {
|
||||
return lb;
|
||||
}
|
||||
}
|
||||
|
||||
return NULL;
|
||||
}
|
||||
|
|
|
@ -44,6 +44,7 @@
|
|||
|
||||
#include "RE_pipeline.h"
|
||||
|
||||
#include "SEQ_channels.h"
|
||||
#include "SEQ_effects.h"
|
||||
#include "SEQ_proxy.h"
|
||||
#include "SEQ_relations.h"
|
||||
|
@ -2421,8 +2422,6 @@ static ImBuf *do_multicam(const SeqRenderData *context,
|
|||
{
|
||||
ImBuf *out;
|
||||
Editing *ed;
|
||||
ListBase *seqbasep;
|
||||
ListBase *channels = &seq->channels;
|
||||
|
||||
if (seq->multicam_source == 0 || seq->multicam_source >= seq->machine) {
|
||||
return NULL;
|
||||
|
@ -2432,7 +2431,8 @@ static ImBuf *do_multicam(const SeqRenderData *context,
|
|||
if (!ed) {
|
||||
return NULL;
|
||||
}
|
||||
seqbasep = SEQ_get_seqbase_by_seq(&ed->seqbase, seq);
|
||||
ListBase *seqbasep = SEQ_get_seqbase_by_seq(&ed->seqbase, seq);
|
||||
ListBase *channels = SEQ_get_channels_by_seq(&ed->seqbase, seq);
|
||||
if (!seqbasep) {
|
||||
return NULL;
|
||||
}
|
||||
|
@ -2463,13 +2463,12 @@ static int early_out_adjustment(Sequence *UNUSED(seq), float UNUSED(fac))
|
|||
static ImBuf *do_adjustment_impl(const SeqRenderData *context, Sequence *seq, float timeline_frame)
|
||||
{
|
||||
Editing *ed;
|
||||
ListBase *seqbasep;
|
||||
ListBase *channels = &seq->channels;
|
||||
ImBuf *i = NULL;
|
||||
|
||||
ed = context->scene->ed;
|
||||
|
||||
seqbasep = SEQ_get_seqbase_by_seq(&ed->seqbase, seq);
|
||||
ListBase *seqbasep = SEQ_get_seqbase_by_seq(&ed->seqbase, seq);
|
||||
ListBase *channels = SEQ_get_channels_by_seq(&ed->seqbase, seq);
|
||||
|
||||
/* Clamp timeline_frame to strip range so it behaves as if it had "still frame" offset (last
|
||||
* frame is static after end of strip). This is how most strips behave. This way transition
|
||||
|
|
Loading…
Reference in New Issue