Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

crash when merging #4844

Closed
2 of 7 tasks
xcstream opened this issue Oct 26, 2017 · 3 comments
Closed
2 of 7 tasks

crash when merging #4844

xcstream opened this issue Oct 26, 2017 · 3 comments
Labels
status: needs feedback Tell me more about it

Comments

@xcstream
Copy link

xcstream commented Oct 26, 2017

  • Gogs version (or commit ref): 0.10.18.0313
  • Git version: 2.7.4
  • Operating system: Ubuntu 16.04
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gogs.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist (usually found in log/gogs.log):

Description

...
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0xb557b6]

goroutine 6734 [running]:
github.com/gogits/gogs/models.(*Repository).getOwner(0x0, 0x16f0600, 0xc420283d60, 0xc4216969c0, 0x36)
/home/vagrant/gopath/src/github.com/gogits/gogs/models/repo.go:273 +0x26
github.com/gogits/gogs/models.(*Repository).mustOwner(0x0, 0x16f0600, 0xc420283d60, 0xc4212703a8)
/home/vagrant/gopath/src/github.com/gogits/gogs/models/repo.go:286 +0x43
github.com/gogits/gogs/models.(*Repository).repoPath(0x0, 0x16f0600, 0xc420283d60, 0x2, 0x2)
/home/vagrant/gopath/src/github.com/gogits/gogs/models/repo.go:415 +0x43
github.com/gogits/gogs/models.(*Repository).RepoPath(0x0, 0x44, 0xc421689d08)
/home/vagrant/gopath/src/github.com/gogits/gogs/models/repo.go:419 +0x43
github.com/gogits/gogs/models.(*PullRequest).PushToBaseRepo(0xc421270380, 0x0, 0x0)
/home/vagrant/gopath/src/github.com/gogits/gogs/models/pull.go:572 +0x152
github.com/gogits/gogs/models.addHeadRepoTasks(0xc4222ffeb0, 0x1, 0x2)
/home/vagrant/gopath/src/github.com/gogits/gogs/models/pull.go:652 +0x165
github.com/gogits/gogs/models.AddTestPullRequestTask(0xc420178600, 0xc, 0xc421945570, 0x7, 0xc42183e500)
/home/vagrant/gopath/src/github.com/gogits/gogs/models/pull.go:698 +0x709
created by github.com/gogits/gogs/models.(*PullRequest).Merge.func1
/home/vagrant/gopath/src/github.com/gogits/gogs/models/pull.go:190 +0xc3

@pgleghorn
Copy link

pgleghorn commented Nov 21, 2017

This was reported to me also, with same stack trace (ending in getOwner), though I didn't witness the exact replication steps. Do you know what steps caused it in your case?
I am using 0_11_33 (SQLlite) on Win2012.

@unknwon
Copy link
Member

unknwon commented Nov 21, 2017

@pgleghorn can you give more info in the form of formal issue report?

@unknwon unknwon added the status: needs feedback Tell me more about it label Nov 21, 2017
@pgleghorn
Copy link

Have raised #4883 for my report, thx.

@unknwon unknwon closed this as completed Nov 22, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: needs feedback Tell me more about it
Projects
None yet
Development

No branches or pull requests

3 participants