Bug 7230 - NULL pointer derref crash in spoolss_addprinterex_level_2
Summary: NULL pointer derref crash in spoolss_addprinterex_level_2
Status: RESOLVED FIXED
Alias: None
Product: Samba 3.3
Classification: Unclassified
Component: Printing (show other bugs)
Version: 3.3.12
Hardware: Other Linux
: P3 normal
Target Milestone: ---
Assignee: Karolin Seeger
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-09 20:04 UTC by Guenther Deschner
Modified: 2010-03-24 08:05 UTC (History)
0 users

See Also:

Attachments
patch for 3.3 (939 bytes, patch)
2010-03-09 20:05 UTC, Guenther Deschner 		no flags Details
patch for 3.3.x (full version) (2.58 KB, patch)
2010-03-12 05:09 UTC, Guenther Deschner 		no flags Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Guenther Deschner 2010-03-09 20:04:33 UTC 
there is a NULL pointer derref crash in spoolss_addprinterex_level_2 which has been fixed in master a long while ago.
Comment 1 Guenther Deschner 2010-03-09 20:05:55 UTC 
Created attachment 5479 [details]
patch for 3.3

patch that went into master sometime last summer
Comment 2 Jeremy Allison 2010-03-10 18:45:34 UTC 
+1 from Jeremy. This is an obvious fix.

Re-assigning to Karolin for inclusion.

Jeremy.
Comment 3 Karolin Seeger 2010-03-11 03:12:56 UTC 
I am sorry, but 3.3 is already in the security fixes only modus.
Closing out bug report.

Please feel free to reopen if you think we should do another 3.3 release due to this issue.

Thanks!
Comment 4 Guenther Deschner 2010-03-12 05:09:07 UTC 
Created attachment 5494 [details]
patch for 3.3.x (full version)

previous version was not fully fixing the crash
Comment 5 Guenther Deschner 2010-03-12 07:18:15 UTC 
Karolin, I am reopening, not for proposing a new release because of this, just to make it easier for vendors to find this bug and the necessary fix.
Comment 6 Karolin Seeger 2010-03-24 08:05:08 UTC 
Closing out bug report after talking to Günther.
It's fixed in current versions, so leaving it open is a bit confusing.