search

thegenemyers / DAMAPPER

Long read to reference genome mapping tool
Other
13 stars 9 forks source link

Core dump in Close_Track #10

Open a-ludi opened 4 years ago

a-ludi commented 4 years ago

In commit 3a4c0c7667f88f3aaffa51688ced9d337f4e233e you introduced a unchecked call to free in DB.c. In damapper it might be called with an argument of record->alen = 0xffffffffffffffff which, of course, leads to a segfault. I am not sure where the value originates but it is always the same value.

thegenemyers commented 4 years ago

Can you please tell me exactly which call to free is problematic? -- Gene (I believe the alen = -1 denotes not used/allocated).

On 3/2/20, 5:41 PM, Arne wrote:

In commit 3a4c0c7 https://github.com/thegenemyers/DAMAPPER/commit/3a4c0c7667f88f3aaffa51688ced9d337f4e233e you introduced a unchecked call to |free| in DB.c https://github.com/thegenemyers/DAMAPPER/commit/3a4c0c7667f88f3aaffa51688ced9d337f4e233e#diff-44d8d21fd7aa68a8169cd8710fa19de2R2079. In |damapper| it might be called with an argument of |record->alen = 0xffffffffffffffff| which, of course, leads to a segfault. I am not sure where the value originates but it is always the same value.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/thegenemyers/DAMAPPER/issues/10?email_source=notifications&email_token=ABUSINT5DJRXQES35NNE7UDRFPOU3A5CNFSM4K7YDFA2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IRYCW5Q, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABUSINXWOW3FPKFR3WUYUDLRFPOU3ANCNFSM4K7YDFAQ.

a-ludi commented 4 years ago

Sorry, I ment to link it. It's in DB.c line 2079.