Re: [tor-bugs] #26467 [Core Tor/Tor]: Coverity issues for week 25

[Tor Bug Tracker & Wiki]

#26467: Coverity issues for week 25
--+--
 Reporter:  ahf   |  Owner:  (none)
 Type:  defect| Status:  closed
 Priority:  High  |  Milestone:  Tor: unspecified
Component:  Core Tor/Tor  |Version:  Tor: unspecified
 Severity:  Normal| Resolution:  fixed
 Keywords:|  Actual Points:
Parent ID:| Points:
 Reviewer:  dgoulet   |Sponsor:
--+--
Changes (by nickm):

 * status:  merge_ready => closed
 * resolution:   => fixed


Comment:

 Merged this to 0.2.9 and forward!

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #26467 [Core Tor/Tor]: Coverity issues for week 25

[Tor Bug Tracker & Wiki]

#26467: Coverity issues for week 25
--+--
 Reporter:  ahf   |  Owner:  (none)
 Type:  defect| Status:  merge_ready
 Priority:  High  |  Milestone:  Tor: unspecified
Component:  Core Tor/Tor  |Version:  Tor: unspecified
 Severity:  Normal| Resolution:
 Keywords:|  Actual Points:
Parent ID:| Points:
 Reviewer:  dgoulet   |Sponsor:
--+--

Comment (by ahf):

 I created #26521 to track that issue.

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #26467 [Core Tor/Tor]: Coverity issues for week 25

[Tor Bug Tracker & Wiki]

#26467: Coverity issues for week 25
--+--
 Reporter:  ahf   |  Owner:  (none)
 Type:  defect| Status:  merge_ready
 Priority:  High  |  Milestone:  Tor: unspecified
Component:  Core Tor/Tor  |Version:  Tor: unspecified
 Severity:  Normal| Resolution:
 Keywords:|  Actual Points:
Parent ID:| Points:
 Reviewer:  dgoulet   |Sponsor:
--+--
Changes (by dgoulet):

 * status:  needs_review => merge_ready


Comment:

 Patches lgtm;

 @ahf, can you point to the bug about the HS service you describe so I can
 take a look at it?

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #26467 [Core Tor/Tor]: Coverity issues for week 25

[Tor Bug Tracker & Wiki]

#26467: Coverity issues for week 25
--+--
 Reporter:  ahf   |  Owner:  (none)
 Type:  defect| Status:  needs_review
 Priority:  High  |  Milestone:  Tor: unspecified
Component:  Core Tor/Tor  |Version:  Tor: unspecified
 Severity:  Normal| Resolution:
 Keywords:|  Actual Points:
Parent ID:| Points:
 Reviewer:  dgoulet   |Sponsor:
--+--
Changes (by asn):

 * reviewer:   => dgoulet


--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #26467 [Core Tor/Tor]: Coverity issues for week 25

[Tor Bug Tracker & Wiki]

#26467: Coverity issues for week 25
--+--
 Reporter:  ahf   |  Owner:  (none)
 Type:  defect| Status:  needs_review
 Priority:  High  |  Milestone:  Tor: unspecified
Component:  Core Tor/Tor  |Version:  Tor: unspecified
 Severity:  Normal| Resolution:
 Keywords:|  Actual Points:
Parent ID:| Points:
 Reviewer:|Sponsor:
--+--
Changes (by ahf):

 * status:  new => needs_review


Comment:

 Patches in the following branches:

 - https://github.com/ahf/tor/tree/maint-0.2.9
 - https://github.com/ahf/tor/tree/maint-0.3.0
 - https://github.com/ahf/tor/tree/maint-0.3.1
 - https://github.com/ahf/tor/tree/maint-0.3.2

 We still need to solve the issue with the defects around the HS `service`
 variable and the `if (BUG(x == NULL)) { ...; return y; }` situation.

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

[tor-bugs] #26467 [Core Tor/Tor]: Coverity issues for week 25

[Tor Bug Tracker & Wiki]

#26467: Coverity issues for week 25
--+--
 Reporter:  ahf   |  Owner:  (none)
 Type:  defect| Status:  new
 Priority:  High  |  Milestone:  Tor: unspecified
Component:  Core Tor/Tor  |Version:  Tor: unspecified
 Severity:  Normal|   Keywords:
Actual Points:|  Parent ID:
   Points:|   Reviewer:
  Sponsor:|
--+--
 - CID 1437460: Fixed: Patch in commit
 e13eac62357d810996a53ff1b5c3532096935833 in the branch coverity-
 week-25-maint-0.3.1.
 - CID 1437459: Not fixed: Odd issue: `if (BUG(x == NULL)) { ... x leaks
 here? ...; return y; }`
 - CID 1437458: Not fixed: Odd issue: `if (BUG(x == NULL)) { ... x leaks
 here? ...; return y; }`
 - CID 1437457: Not fixed: The service variable is free'd in hs_free_all()
 - is this to "deeply" hidden away from Coverity to spot? Calling
 `hs_service_free(service)` before the `hs_free_all()` leads to a segfault.
 - CID 1437456: Fixed: Should be fixed by the changes made for CID 1437460.
 - CID 1437455: Not fixed: Same problem as in 1437457, but for
 `test_desc_reupload_logic()`.
 - CID 1437454: Fixed: Patch in commit
 a2d6e0b16a254913275f4536e2796ac5a59d2fee in the branch coverity-
 week-25-maint-0.3.3.
 - CID 1437453: Fixed: Patch in commit
 1724f995c71502977c79343922ab50fc2403284f in the branch coverity-
 week-25-maint-0.2.9. Note that this have existed for longer, but 0.2.9 is
 our oldest maintained version of Tor.
 - CID 1437452: Not fixed: Same problem as in 1437457, but for
 `test_service_event()`.
 - CID 1437451: Fixed: Patch in commit
 3525267b83fd0030929c407ff94f30dbd5b4bd9f in the branch coverity-
 week-25-maint-0.2.9.
 - CID 1437450: Fixed: This should be fixed by the patch for CID 14377460
 in commit e13eac62357d810996a53ff1b5c3532096935833, but needs to be ported
 to the maint-0.3.2 branch.
 - CID 1437449: Fixed: Patch in commit
 ce5d055ed7e4d8e5d1c0b0b922738c87b6a8da1c in the branch coverity-
 week-25-maint-0.3.2.
 - CID 1437448: Fixed: Patch in commit
 dc2384da30cae716f512dedef37d27f00c43f29d in the branch coverity-
 week-25-maint-0.3.1.
 - CID 1437447: Fixed: Patch in commit
 d86c45bf5cc75a526b884a754d72ef4d11aa0693 in the branch coverity-
 week-25-maint-0.3.1.
 - CID 1437446: Not fixed. Same problem as in 1437457, but for
 `test_closing_intro_circs()`.
 - CID 1437445: Fixed: Patch in commit
 8e805bf0f65ffd137a6c9ed89be060e5d9ba317d in the branch coverity-
 week-25-maint-0.3.2.
 - CID 1437444: Not fixed: Odd issue: `if (BUG(x == NULL)) { ... x leaks
 here? ...; return y; }`
 - CID 1437443: Not fixed. Same problem as in 1437457, but for
 `test_introduce2()`.
 - CID 1437442: Not fixed: Same issue as with the `service` variables, but
 with a channel_listener_t.
 - CID 1437441: Fixed: Patch in commit
 f57789c994a499fbb9bb1bb8f37599abe297 in the branch coverity-
 week-25-maint-0.2.9.
 - CID 1437440: Fixed: Patch in commit
 b35fc51a2e6bd72707c88fa5067ab90e837b0c03 in the branch coverity-
 week-25-maint-0.2.9.
 - CID 1437439: Not fixed: Odd issue: `if (BUG(x == NULL)) { ... x leaks
 here? ...; return y; }`
 - CID 1437438: Fixed: Patch in commit
 d8230d8ba2f11bc060862f88efd49ae3178c3c5a in the branch coverity-
 week-25-maint-0.3.3.
 - CID 1437437: Fixed: Patch in commit
 3d80c086bea3b0d93327c30ac620740b629cb294 in the branch coverity-
 week-25-maint-0.3.0.
 - CID 1437436: Fixed: Patch in commit
 3b6c93782d97839ced3f57d0cea3904c200da6ed in the branch coverity-
 week-25-maint-0.3.3.
 - CID 1437435: Not fixed. Same problem as in 1437457, but for
 `test_closing_intro_circs()`.
 - CID 1437434: Fixed: Patch in commit
 c997d49ad6e6a360323311444eed1dbee785aea4 in the branch coverity-
 week-25-maint-0.3.0.
 - CID 1437433: Not fixed: Odd issue: `if (BUG(x == NULL)) { ... x leaks
 here? ...; return y; }`
 - CID 1437432: Not fixed. Same problem as in 1437457, but for
 `test_helper_functions()`.
 - CID 1437431: Fixed: Patch in commit
 8550016e6f5e2780259f5073e34e67708e4e87ff in the branch coverity-
 week-25-maint-0.3.1.
 - CID 1437430: Not fixed. Same problem as in 1437457, but for
 `test_rotate_descriptors()`.
 - CID 1437429: Fixed: Should be fixed by the patch for CID 1437454.
 - CID 1437428: Not fixed. Same problem as in 1437457, but for
 `test_build_update_descriptors()`.
 - CID 1437427: Fixed: patch in commit
 a2e623f631a63f041f76b79e554b01bbf62748b8 in the branch coverity-
 week-25-maint-0.3.2.
 - CID 1437426: Not fixed. Same problem as in 1437457, but for
 `test_intro_established()`.

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs