Am 17.08.18 um 14:51 schrieb Carl Eugen Hoyos:
> 2018-08-17 14:36 GMT+02:00, Hendrik Leppkes :
>> On Fri, Aug 17, 2018 at 12:05 PM Paweł Wegner
>> wrote:
>>>
>>> Yes, it will work.
>>>
>>> Fixes: runtime error: passing uninitialized value to FreeContextBuffer
>>> causes a crash
>>>
>>>
2018-08-17 14:36 GMT+02:00, Hendrik Leppkes :
> On Fri, Aug 17, 2018 at 12:05 PM Paweł Wegner
> wrote:
>>
>> Yes, it will work.
>>
>> Fixes: runtime error: passing uninitialized value to FreeContextBuffer
>> causes a crash
>>
>> Signed-off-by: Paweł Wegner
>> ---
>> libavformat/tls_schannel.c |
On Fri, Aug 17, 2018 at 12:05 PM Paweł Wegner wrote:
>
> Yes, it will work.
>
> Fixes: runtime error: passing uninitialized value to FreeContextBuffer
> causes a crash
>
> Signed-off-by: Paweł Wegner
> ---
> libavformat/tls_schannel.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>
Yes, it will work.
Fixes: runtime error: passing uninitialized value to FreeContextBuffer
causes a crash
Signed-off-by: Paweł Wegner
---
libavformat/tls_schannel.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/tls_schannel.c b/libavformat/tls_schannel.c
index
Yes, this works as well. I attached fixed patch.
On Fri, Aug 17, 2018 at 11:15 AM Carl Eugen Hoyos
wrote:
> 2018-08-03 20:28 GMT+02:00, Paweł Wegner :
>
> > One could copy the initialization to the top from the while loop:
> > init_sec_buffer([0], SECBUFFER_TOKEN, NULL, 0);
> >
2018-08-03 20:28 GMT+02:00, Paweł Wegner :
> One could copy the initialization to the top from the while loop:
> init_sec_buffer([0], SECBUFFER_TOKEN, NULL, 0);
> init_sec_buffer([1], SECBUFFER_ALERT, NULL, 0);
> init_sec_buffer([2], SECBUFFER_EMPTY, NULL, 0);
>
Could someone merge it?
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
One could copy the initialization to the top from the while loop:
init_sec_buffer([0], SECBUFFER_TOKEN, NULL, 0);
init_sec_buffer([1], SECBUFFER_ALERT, NULL, 0);
init_sec_buffer([2], SECBUFFER_EMPTY, NULL, 0);
init_sec_buffer_desc(_desc, outbuf, 3);
But memset is shorter. Current code
One could copy the initialization to the top from the while loop:
init_sec_buffer([0], SECBUFFER_TOKEN, NULL, 0);
init_sec_buffer([1], SECBUFFER_ALERT, NULL, 0);
init_sec_buffer([2], SECBUFFER_EMPTY, NULL, 0);
init_sec_buffer_desc(_desc, outbuf, 3);
But memset is shorter. Current code
2018-08-02 20:46 GMT+02:00, Paweł Wegner :
> Fixes: runtime error: passing uninitialized value to FreeContextBuffer
> causes a crash
> ---
> libavformat/tls_schannel.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/libavformat/tls_schannel.c b/libavformat/tls_schannel.c
> index
Fixes: runtime error: passing uninitialized value to FreeContextBuffer
causes a crash
Signed-off-by: Paweł Wegner
---
libavformat/tls_schannel.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/libavformat/tls_schannel.c b/libavformat/tls_schannel.c
index f41b007773..6b51c1f3dc 100644
---
Fixes: runtime error: passing uninitialized value to FreeContextBuffer
causes a crash
---
libavformat/tls_schannel.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/libavformat/tls_schannel.c b/libavformat/tls_schannel.c
index 065dccb..6953008 100644
--- a/libavformat/tls_schannel.c
+++
12 matches
Mail list logo