[PR #5999] [MERGED] fix nil deref in auth.go #37859

Closed
opened 2026-04-22 22:31:24 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/5999
Author: @mxyng
Created: 7/26/2024
Status: Merged
Merged: 7/26/2024
Merged by: @mxyng

Base: mainHead: mxyng/fix-push


📝 Commits (1)

📊 Changes

2 files changed (+2 additions, -2 deletions)

View changed files

📝 server/auth.go (+1 -1)
📝 server/upload.go (+1 -1)

📄 Description

server/auth.go uses makeRequest with nil registryOptions so registryOptions.CheckRedirect will dereference nil and panic


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ollama/ollama/pull/5999 **Author:** [@mxyng](https://github.com/mxyng) **Created:** 7/26/2024 **Status:** ✅ Merged **Merged:** 7/26/2024 **Merged by:** [@mxyng](https://github.com/mxyng) **Base:** `main` ← **Head:** `mxyng/fix-push` --- ### 📝 Commits (1) - [`a622c47`](https://github.com/ollama/ollama/commit/a622c47bd32e4c7d8d6cd12ba8c7556fcc492524) fix nil deref in auth.go ### 📊 Changes **2 files changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `server/auth.go` (+1 -1) 📝 `server/upload.go` (+1 -1) </details> ### 📄 Description `server/auth.go` uses `makeRequest` with `nil` `registryOptions` so `registryOptions.CheckRedirect` will dereference `nil` and panic --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-22 22:31:24 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#37859