Opened 9 years ago

Closed 9 years ago

#268 closed defect (fixed)

some memory handling issues

Reported by: kolya Owned by: davidf
Priority: minor Milestone: HM-8.0
Component: HM Version: HM-5.0
Keywords: memory Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

The bitstream was generated with
./TAppEncoder -c ../cfg/encoder_randomaccess.cfg -c ../cfg/per-sequence/BQSQuare.cfg -g 8 -f 17 -q 37

with gcc 4.4.1

Here below is the valgrind memcheck output for decoder run with this bitstream.

Attachments (3)

valgrind_24717.txt (10.4 KB) - added by kolya 9 years ago.
valgrind_28037.txt (6.7 KB) - added by kolya 9 years ago.
valgrind_28213.txt (5.8 KB) - added by kolya 9 years ago.
report for encoder of HM-5.0-dev-bugfix

Download all attachments as: .zip

Change History (11)

Changed 9 years ago by kolya

comment:1 Changed 9 years ago by kolya

  • Keywords memory added
  • Milestone set to HM-5.1
  • Version set to HM-5.0rc1

comment:2 follow-up: Changed 9 years ago by ksuehring

A number of ALF related issues have been fixed between rc1 and HM 5.0.

There are also some fixed added on top of that that might fix some of the issues in HM-5.0-dev-bugfix

Can you please test on that branch again?

Changed 9 years ago by kolya

comment:3 in reply to: ↑ 2 Changed 9 years ago by kolya

Replying to ksuehring:

A number of ALF related issues have been fixed between rc1 and HM 5.0.

There are also some fixed added on top of that that might fix some of the issues in HM-5.0-dev-bugfix

Can you please test on that branch again?

Please take my apologize, I should test bugfix branch of course. Yet some problems remain, see the new report attached.

Bitstream was generated by HM-5.0-dev-bugfix, decoder of the same revision was run with TAppDecoderStaticd -b str.bin.

Report for the encoder is also attached, with 28213 number.

Last edited 9 years ago by kolya (previous) (diff)

Changed 9 years ago by kolya

report for encoder of HM-5.0-dev-bugfix

comment:4 Changed 9 years ago by ksuehring

  • Component changed from CHANGEME to HM
  • Version changed from HM-5.0rc1 to HM-5.0

comment:5 Changed 9 years ago by ksuehring

The RPS related issues have been taken care of in r1742

comment:6 Changed 9 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:7 Changed 9 years ago by davidf

  • Cc jct-vc@… added

comment:8 Changed 9 years ago by ksuehring

  • Milestone changed from HM-5.1 to HM-8.0
  • Resolution set to fixed
  • Status changed from new to closed

I remember having fixed these remaining issues. Should have been at least before HM 8.0.

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • David Flynn(Owner, Subscriber, Participant)
  • Frank Bossen(Subscriber)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)
  • Nikolay Shlyakhov(Reporter, Participant)