Use LEN(dc.col) instead of LEN(colorname).
LEN(colorname) may be below 256 for some configurations.
This commit is contained in:
		
				
					committed by
					
						 Roberto E. Vargas Caballero
						Roberto E. Vargas Caballero
					
				
			
			
				
	
			
			
			
						parent
						
							2fdcc5e5f6
						
					
				
				
					commit
					84c756b97e
				
			
							
								
								
									
										4
									
								
								st.c
									
									
									
									
									
								
							
							
						
						
									
										4
									
								
								st.c
									
									
									
									
									
								
							| @ -2911,7 +2911,7 @@ xloadcols(void) { | |||||||
| 	} | 	} | ||||||
|  |  | ||||||
| 	/* load colors [0-15] and [256-LEN(colorname)] (config.h) */ | 	/* load colors [0-15] and [256-LEN(colorname)] (config.h) */ | ||||||
| 	for(i = 0; i < LEN(colorname); i++) { | 	for(i = 0; i < LEN(dc.col); i++) { | ||||||
| 		if(!colorname[i]) | 		if(!colorname[i]) | ||||||
| 			continue; | 			continue; | ||||||
| 		if(!XftColorAllocName(xw.dpy, xw.vis, xw.cmap, colorname[i], &dc.col[i])) { | 		if(!XftColorAllocName(xw.dpy, xw.vis, xw.cmap, colorname[i], &dc.col[i])) { | ||||||
| @ -2942,7 +2942,7 @@ xsetcolorname(int x, const char *name) { | |||||||
| 	XRenderColor color = { .alpha = 0xffff }; | 	XRenderColor color = { .alpha = 0xffff }; | ||||||
| 	Color ncolor; | 	Color ncolor; | ||||||
|  |  | ||||||
| 	if(!BETWEEN(x, 0, LEN(colorname))) | 	if(!BETWEEN(x, 0, LEN(dc.col))) | ||||||
| 		return 1; | 		return 1; | ||||||
|  |  | ||||||
| 	if(!name) { | 	if(!name) { | ||||||
|  | |||||||
		Reference in New Issue
	
	Block a user